LGTM

http://gwt-code-reviews.appspot.com/1306801/diff/1/2
File user/src/com/google/gwt/editor/client/EditorDelegate.java (right):

http://gwt-code-reviews.appspot.com/1306801/diff/1/2#newcode62
user/src/com/google/gwt/editor/client/EditorDelegate.java:62: * used a a
supplementary signal.
"...however, this calculated dirty state is overridden by explicit calls
to setDirty."

http://gwt-code-reviews.appspot.com/1306801/diff/1/4
File
user/src/com/google/gwt/editor/client/impl/AbstractEditorDelegate.java
(right):

http://gwt-code-reviews.appspot.com/1306801/diff/1/4#newcode255
user/src/com/google/gwt/editor/client/impl/AbstractEditorDelegate.java:255:
* Returns {@code true} if the editor contains leaf editors without
delegates.
shouldn't the name mention leaf editors, then?

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

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

Reply via email to