http://gwt-code-reviews.appspot.com/925801/diff/1/2
File user/src/com/google/gwt/app/place/AbstractProxyListActivity.java
(right):

http://gwt-code-reviews.appspot.com/925801/diff/1/2#newcode256
user/src/com/google/gwt/app/place/AbstractProxyListActivity.java:256: if
(view == null) {
Isn't the problem that this activity isn't reusable? I mean, it
initializes the view in its constructor (lines 87 to 108) and clears
everything in onStop().
Shouldn't it rather initializes in start()? and keep the view around in
onStop so that it can be re-initialized (re-start()ed)? (and probably
clear the idToRow and idToProxy maps)

This check would then be "if (display == null)", just like in
AbstractProxyEditActivity.

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

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

Reply via email to