LGTM, with nits.
http://gwt-code-reviews.appspot.com/703801/diff/1/2 File user/src/com/google/gwt/uibinder/elementparsers/GridParser.java (right): http://gwt-code-reviews.appspot.com/703801/diff/1/2#newcode2 user/src/com/google/gwt/uibinder/elementparsers/GridParser.java:2: * Copyright 2009 Google Inc. Has this code been released before? http://gwt-code-reviews.appspot.com/703801/diff/1/2#newcode84 user/src/com/google/gwt/uibinder/elementparsers/GridParser.java:84: Vector<Vector<CellContent>> matrix = new Vector<Vector<CellContent>>(); Vector and not ArrayList? http://gwt-code-reviews.appspot.com/703801/diff/1/6 File user/test/com/google/gwt/uibinder/elementparsers/GridParserTest.java (right): http://gwt-code-reviews.appspot.com/703801/diff/1/6#newcode52 user/test/com/google/gwt/uibinder/elementparsers/GridParserTest.java:52: } catch (UnableToCompleteException e) { Scott says rename e -> expected for clarity when checking for thrown exceptions. http://gwt-code-reviews.appspot.com/703801/diff/1/7 File user/test/com/google/gwt/uibinder/test/UiJavaResources.java (right): http://gwt-code-reviews.appspot.com/703801/diff/1/7#newcode108 user/test/com/google/gwt/uibinder/test/UiJavaResources.java:108: }; Needs an empty line. http://gwt-code-reviews.appspot.com/703801/diff/1/10 File user/test/com/google/gwt/uibinder/test/client/WidgetBasedUi.java.rej (right): http://gwt-code-reviews.appspot.com/703801/diff/1/10#newcode1 user/test/com/google/gwt/uibinder/test/client/WidgetBasedUi.java.rej:1: *************** Don't commit this file. http://gwt-code-reviews.appspot.com/703801/diff/1/11 File user/test/com/google/gwt/uibinder/test/client/WidgetBasedUi.ui.xml (right): http://gwt-code-reviews.appspot.com/703801/diff/1/11#newcode614 user/test/com/google/gwt/uibinder/test/client/WidgetBasedUi.ui.xml:614: </gwt:customCell> Indentation weirdness. http://gwt-code-reviews.appspot.com/703801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
