LGTM
Now, what's the expected behavior for SafeUri?
- URL_ATTRIBUTE_ENTIRE => OK
- URL_ATTRIBUTE_START => log a warning ? throw ?
- others => OK ? log a warning ? (note that the previously mentioned
URL attributes introduced by HTML5, which include <video poster=> for
instance, will fall in this category; should there be an
ATTRIBUTE_ENTIRE to allow <video poster="{0}"></video> but warn/disallow
<video poster="{0}foo"></video>?)
http://gwt-code-reviews.appspot.com/1396803/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors