http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/src/com/google/gwt/uibinder/rebind/AbstractFieldWriter.java File user/src/com/google/gwt/uibinder/rebind/AbstractFieldWriter.java (right):
http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/src/com/google/gwt/uibinder/rebind/AbstractFieldWriter.java#newcode119 user/src/com/google/gwt/uibinder/rebind/AbstractFieldWriter.java:119: public void resetWritten() { No. http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java File user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java (right): http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java#newcode1360 user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java:1360: fieldManager.rewriteGwtFieldsDeclaration(niceWriter, uiOwnerType.getName()); I called this out in the first review. What's up with this rewriting stuff, and resetting the written status in the field writers? It seems very wrong. http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java#newcode1469 user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java:1469: // TODO(sbrubaker): Fix method signature Stephanie won't be doing these. Shouldn't these issues be fixed now? http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java#newcode1479 user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java:1479: // TODO(sbrubaker): Find a better way to remove "asString" ditto http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/test/com/google/gwt/uibinder/elementparsers/DialogBoxParserTest.java File user/test/com/google/gwt/uibinder/elementparsers/DialogBoxParserTest.java (right): http://gwt-code-reviews.appspot.com/1442804/diff/7001/user/test/com/google/gwt/uibinder/elementparsers/DialogBoxParserTest.java#newcode80 user/test/com/google/gwt/uibinder/elementparsers/DialogBoxParserTest.java:80: "fieldName.setHTML(template.html1().asString());", This is a very fundamental change of the test. You're no longer verifying the output of the parser. http://gwt-code-reviews.appspot.com/1442804/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
