LGTM

with one tweak


http://gwt-code-reviews.appspot.com/720801/diff/13001/9006
File user/src/com/google/gwt/cell/client/ImageLoadingCell.java (right):

http://gwt-code-reviews.appspot.com/720801/diff/13001/9006#newcode123
user/src/com/google/gwt/cell/client/ImageLoadingCell.java:123: this(new
DefaultRenderers());
GWT.create(DefaultRenderers) will allow more flexibility at no cost.
Should tweak javadoc to note it's made this way.

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

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

Reply via email to