LGTM

http://gwt-code-reviews.appspot.com/645802/diff/1/2
File user/src/com/google/gwt/user/client/ui/Tree.java (right):

http://gwt-code-reviews.appspot.com/645802/diff/1/2#newcode1002
user/src/com/google/gwt/user/client/ui/Tree.java:1002: root = new
TreeItem() {
Is there a way to avoid duplicating so much code from
TreeItem.insertItem?  Can the superclass implementation take a flag to
control whatever behavior is different here from the typical case?

http://gwt-code-reviews.appspot.com/645802/diff/1/4
File user/test/com/google/gwt/user/client/ui/TreeItemTest.java (right):

http://gwt-code-reviews.appspot.com/645802/diff/1/4#newcode32
user/test/com/google/gwt/user/client/ui/TreeItemTest.java:32:
typo: 'Readd'

http://gwt-code-reviews.appspot.com/645802/diff/1/4#newcode33
user/test/com/google/gwt/user/client/ui/TreeItemTest.java:33: // Readd
the only item.
typo: 'Readd'

http://gwt-code-reviews.appspot.com/645802/diff/1/4#newcode39
user/test/com/google/gwt/user/client/ui/TreeItemTest.java:39: // Readd
at the end.
typo: 'Readd'

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

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

Reply via email to