Adding a couple of useful methods to AbstractCellTable.  getColumnWidth() can be used to access the width of a column set using setColumnWidth().  flush() will immediately flush pending changes into the view.

Review at http://gwt-code-reviews.appspot.com/1453801

Review by: pengzhuang@google.com

git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@10259 8db76d5a-ed1c-0410-87a9-c151d255dfc7

user/src/com/google/gwt/user/cellview/client/AbstractCellTable.java

diff --git a/user/src/com/google/gwt/user/cellview/client/AbstractCellTable.java b/user/src/com/google/gwt/user/cellview/client/AbstractCellTable.java
index 019f4d4..169d915 100644
--- a/user/src/com/google/gwt/user/cellview/client/AbstractCellTable.java
+++ b/user/src/com/google/gwt/user/cellview/client/AbstractCellTable.java
@@ -635,6 +635,20 @@
   }
 
   /**
+   * Flush all pending changes to the table and render immediately.
+   * 
+   * <p>
+   * Modifications to the table, such as adding columns or setting data, are not
+   * rendered immediately. Instead, changes are coalesced at the end of the
+   * current event loop to avoid rendering the table multiple times. Use this
+   * method to force the table to render all pending modifications immediately.
+   * </p>
+   */
+  public void flush() {
+    getPresenter().flush();
+  }
+
+  /**
    * Get the column at the specified index.
    * 
    * @param col the index of the column to retrieve
@@ -676,6 +690,17 @@
   }
 
   /**
+   * Get the width of a {@link Column}.
+   * 
+   * @param column the column
+   * @return the width of the column, or null if not set
+   * @see #setColumnWidth(Column, double, Unit)
+   */
+  public String getColumnWidth(Column<T, ?> column) {
+    return columnWidths.get(column);
+  }
+
+  /**
    * Get the widget displayed when the table has no rows.
    * 
    * @return the empty table widget
@@ -703,7 +728,7 @@
    *           current page
    */
   public TableRowElement getRowElement(int row) {
-    getPresenter().flush();
+    flush();
     checkRowBounds(row);
     NodeList<TableRowElement> rows = getTableBodyElement().getRows();
     return rows.getLength() > row ? rows.getItem(row) : null;

user/test/com/google/gwt/user/cellview/client/AbstractCellTableTestBase.java

diff --git a/user/test/com/google/gwt/user/cellview/client/AbstractCellTableTestBase.java b/user/test/com/google/gwt/user/cellview/client/AbstractCellTableTestBase.java
index 59f3178..185ff86 100644
--- a/user/test/com/google/gwt/user/cellview/client/AbstractCellTableTestBase.java
+++ b/user/test/com/google/gwt/user/cellview/client/AbstractCellTableTestBase.java
@@ -21,6 +21,7 @@
 import com.google.gwt.dom.client.Document;
 import com.google.gwt.dom.client.NativeEvent;
 import com.google.gwt.dom.client.TableCellElement;
+import com.google.gwt.dom.client.Style.Unit;
 import com.google.gwt.safehtml.shared.SafeHtmlBuilder;
 import com.google.gwt.user.cellview.client.LoadingStateChangeEvent.LoadingState;
 import com.google.gwt.user.client.ui.HasHorizontalAlignment;
@@ -272,6 +273,28 @@
     }
   }
 
+  public void testSetColumnWidth() {
+    AbstractCellTable<String> table = createAbstractHasData(new TextCell());
+    Column<String, ?> col0 = new MockColumn<String, String>();
+    Column<String, ?> col1 = new MockColumn<String, String>();
+    Column<String, ?> col2 = new MockColumn<String, String>();
+
+    // Set a column width.
+    table.setColumnWidth(col0, "100px");
+    table.setColumnWidth(col1, 200.0, Unit.EM);
+    assertEquals("100px", table.getColumnWidth(col0));
+
+    // Some browsers return 200.0, others 200.
+    assertTrue(table.getColumnWidth(col1).contains("200"));
+
+    // Check a column that has not been set.
+    assertNull(table.getColumnWidth(col2));
+
+    // Check a column that has been cleared.
+    table.clearColumnWidth(col0);
+    assertNull(table.getColumnWidth(col0));
+  }
+
   public void testSetEmptyTableWidget() {
     AbstractCellTable<String> table = createAbstractHasData(new TextCell());