Simplified test, and added that enum. One more look, please?
http://gwt-code-reviews.appspot.com/78808/diff/1/2 File user/src/com/google/gwt/uibinder/rebind/BundleWriter.java (right): http://gwt-code-reviews.appspot.com/78808/diff/1/2#newcode90 Line 90: On 2009/10/12 17:53:12, bobv wrote: > Should be zero-length. Done. http://gwt-code-reviews.appspot.com/78808/diff/1/3 File user/src/com/google/gwt/uibinder/rebind/UiBinderParser.java (right): http://gwt-code-reviews.appspot.com/78808/diff/1/3#newcode233 Line 233: if ("with".equals(localName)) { Nice idea, thanks. Done. http://gwt-code-reviews.appspot.com/78808/diff/1/4 File user/src/com/google/gwt/uibinder/rebind/model/ImplicitClientBundle.java (right): http://gwt-code-reviews.appspot.com/78808/diff/1/4#newcode120 Line 120: On 2009/10/12 17:53:12, bobv wrote: > Trailing spaces. Done. http://gwt-code-reviews.appspot.com/78808/diff/1/11 File user/test/com/google/gwt/uibinder/sample/client/UiBinderTest.java (right): http://gwt-code-reviews.appspot.com/78808/diff/1/11#newcode361 Line 361: public void testDataResource() { On 2009/10/12 17:53:12, bobv wrote: > This test isn't right. On IE6 in https, we can't use mhtml and we might have to > pull mhtml support for IE7+Vista if the spriting issues can't be solved. Done. http://gwt-code-reviews.appspot.com/78808 --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
