[gwt-contrib] Re: Make ListEditorWrapper update its backing on flush(). (issue1159801)

2010-11-30 Thread bobv
LGTM, nits. http://gwt-code-reviews.appspot.com/1159801/diff/1/2 File user/src/com/google/gwt/editor/client/adapters/ListEditorWrapper.java (right): http://gwt-code-reviews.appspot.com/1159801/diff/1/2#newcode33 user/src/com/google/gwt/editor/client/adapters/ListEditorWrapper.java:33: private

[gwt-contrib] Re: Make ListEditorWrapper update its backing on flush(). (issue1159801)

2010-11-30 Thread rchandia
http://gwt-code-reviews.appspot.com/1159801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Make ListEditorWrapper update its backing on flush(). (issue1159801)

2010-11-30 Thread rchandia
http://gwt-code-reviews.appspot.com/1159801/diff/1/2 File user/src/com/google/gwt/editor/client/adapters/ListEditorWrapper.java (right): http://gwt-code-reviews.appspot.com/1159801/diff/1/2#newcode33 user/src/com/google/gwt/editor/client/adapters/ListEditorWrapper.java:33: private final ListT

[gwt-contrib] Re: Make ListEditorWrapper update its backing on flush(). (issue1159801)

2010-11-30 Thread rchandia
Commited as of r9305 On 2010/11/30 17:31:54, rchandia wrote: http://gwt-code-reviews.appspot.com/1159801/diff/1/2 File user/src/com/google/gwt/editor/client/adapters/ListEditorWrapper.java (right): http://gwt-code-reviews.appspot.com/1159801/diff/1/2#newcode33