LGTM


http://gwt-code-reviews.appspot.com/1606804/diff/2002/user/src/com/google/gwt/safehtml/shared/SafeHtmlHostedModeUtils.java
File
user/src/com/google/gwt/safehtml/shared/SafeHtmlHostedModeUtils.java
(right):

http://gwt-code-reviews.appspot.com/1606804/diff/2002/user/src/com/google/gwt/safehtml/shared/SafeHtmlHostedModeUtils.java#newcode75
user/src/com/google/gwt/safehtml/shared/SafeHtmlHostedModeUtils.java:75:
* For example, this check will pass for the following strings:
It seems like the first half of the JavaDoc for this method should be
moved to the other one, now that it's public? For
maybeCheckCompleteHtml(), we can just document the conditions under
which it will do the check.

http://gwt-code-reviews.appspot.com/1606804/

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

Reply via email to