D'oh, where was I when Element#setInnerSafeHtml was added? Didn't know
it even existed!

LGTM++

Except you missed one that spans 2 lines in ImageLoadingCell.
(caught using "grep -R -A 1 'setInnerHTML' user/src/com/google/gwt/
|grep asString |grep -v setInnerHTML" ;-) )

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

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

Reply via email to