Thanks for the patch, Thomas.

Yeah, I hate to say this but could you make the fix backward compatible?



http://gwt-code-reviews.appspot.com/642801/diff/1/3
File
user/test/com/google/gwt/uibinder/rebind/model/OwnerFieldClassTest.java
(right):

http://gwt-code-reviews.appspot.com/642801/diff/1/3#newcode62
user/test/com/google/gwt/uibinder/rebind/model/OwnerFieldClassTest.java:62:
assertMethod(htmlSetter, "setHTML",
gwtTypeAdapter.adaptJavaClass(String.class));
Rather than completely replacing the test for a conventionally named
field, could you make the HTML test an additional one? That should be as
simple as repeating these last few lines for "setText"

http://gwt-code-reviews.appspot.com/642801/show

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to