LGTM
http://gwt-code-reviews.appspot.com/1385807/diff/1/user/src/com/google/gwt/user/cellview/client/HasDataPresenter.java File user/src/com/google/gwt/user/cellview/client/HasDataPresenter.java (right): http://gwt-code-reviews.appspot.com/1385807/diff/1/user/src/com/google/gwt/user/cellview/client/HasDataPresenter.java#newcode954 user/src/com/google/gwt/user/cellview/client/HasDataPresenter.java:954: * Protected so that subclasses can override to use an alternative Can you add a <p> before this line so it formats correctly in JavaDoc. http://gwt-code-reviews.appspot.com/1385807/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors