http://gwt-code-reviews.appspot.com/829801/diff/7001/8002 File user/src/com/google/gwt/user/User.gwt.xml (right):
http://gwt-code-reviews.appspot.com/829801/diff/7001/8002#newcode51 user/src/com/google/gwt/user/User.gwt.xml:51: <inherits name="com.google.gwt.safehtml.SafeHtml" /> Maybe alphabetize? http://gwt-code-reviews.appspot.com/829801/diff/7001/8004 File user/src/com/google/gwt/user/client/ui/CheckBox.java (right): http://gwt-code-reviews.appspot.com/829801/diff/7001/8004#newcode70 user/src/com/google/gwt/user/client/ui/CheckBox.java:70: * Creates a check box with the specified text label. "text" -> "html" or "safe html" http://gwt-code-reviews.appspot.com/829801/diff/7001/8005 File user/src/com/google/gwt/user/client/ui/HTML.java (right): http://gwt-code-reviews.appspot.com/829801/diff/7001/8005#newcode45 user/src/com/google/gwt/user/client/ui/HTML.java:45: public class HTML extends Label implements HasDirectionalSafeHtml { Do you mean to remove HasDirectionalHTML? In general, changes to interfaces can cause a lot of compatibility problems -- is there a strategy on how to deprecate the raw HTML interfaces without breaking all users at once? http://gwt-code-reviews.appspot.com/829801/diff/7001/8005#newcode85 user/src/com/google/gwt/user/client/ui/HTML.java:85: setHTML(html.asString()); Shouldn't this call some variant of setSafeHtml? http://gwt-code-reviews.appspot.com/829801/diff/7001/8010 File user/src/com/google/gwt/user/client/ui/MenuItem.java (right): http://gwt-code-reviews.appspot.com/829801/diff/7001/8010#newcode42 user/src/com/google/gwt/user/client/ui/MenuItem.java:42: * Similar to {...@link #MenuItem(String, boolean)} Maybe change this to: Similar to {...@link #MenuItem(String, boolean) MenuItem(String, true)} to indicate that the effect is like setting the flag to "true" http://gwt-code-reviews.appspot.com/829801/diff/7001/8010#newcode227 user/src/com/google/gwt/user/client/ui/MenuItem.java:227: Spaces http://gwt-code-reviews.appspot.com/829801/diff/7001/8010#newcode231 user/src/com/google/gwt/user/client/ui/MenuItem.java:231: Spaces http://gwt-code-reviews.appspot.com/829801/diff/7001/8010#newcode253 user/src/com/google/gwt/user/client/ui/MenuItem.java:253: Spaces http://gwt-code-reviews.appspot.com/829801/diff/7001/8011 File user/src/com/google/gwt/user/client/ui/RadioButton.java (right): http://gwt-code-reviews.appspot.com/829801/diff/7001/8011#newcode77 user/src/com/google/gwt/user/client/ui/RadioButton.java:77: * @param label this radio button's label as SafeHtml http://gwt-code-reviews.appspot.com/829801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
