http://gwt-code-reviews.appspot.com/1355805/diff/6003/user/src/com/google/gwt/user/client/ui/DeckPanel.java
File user/src/com/google/gwt/user/client/ui/DeckPanel.java (right):

http://gwt-code-reviews.appspot.com/1355805/diff/6003/user/src/com/google/gwt/user/client/ui/DeckPanel.java#newcode108
user/src/com/google/gwt/user/client/ui/DeckPanel.java:108: // Figure out
if the deck panel has a fixed height
I'm not sure I'm a good checker, but it smells funny to me that onStart
is losing a check that it used to have without (that I saw) routing
through the new code in newWidget, nor why newWidget, having decided to
animate, doesn't touch onStart.

I'm willing to believe there's a good reason for it, but I'd like to
know what it is. ;-)

http://gwt-code-reviews.appspot.com/1355805/

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

Reply via email to