http://gwt-code-reviews.appspot.com/1340803/diff/1/12 File user/src/com/google/gwt/user/client/ui/DeckLayoutPanel.java (right):
http://gwt-code-reviews.appspot.com/1340803/diff/1/12#newcode28 user/src/com/google/gwt/user/client/ui/DeckLayoutPanel.java:28: * {@link com.google.gwt.user.client.ui.TabPanel}. Do you mean TabPanel or TabLayoutPanel? http://gwt-code-reviews.appspot.com/1340803/diff/1/21 File user/test/com/google/gwt/user/client/ui/DeckLayoutPanelTest.java (right): http://gwt-code-reviews.appspot.com/1340803/diff/1/21#newcode32 user/test/com/google/gwt/user/client/ui/DeckLayoutPanelTest.java:32: // Show widget at index 1, make sure it becomes visible. You may want to note that you're testing both forms of showWidget (here and below). http://gwt-code-reviews.appspot.com/1340803/diff/1/22 File user/test/com/google/gwt/user/client/ui/TabLayoutPanelTest.java (right): http://gwt-code-reviews.appspot.com/1340803/diff/1/22#newcode269 user/test/com/google/gwt/user/client/ui/TabLayoutPanelTest.java:269: assertEquals("inserted text", Extra spaces after this line. http://gwt-code-reviews.appspot.com/1340803/diff/1/22#newcode271 user/test/com/google/gwt/user/client/ui/TabLayoutPanelTest.java:271: assertEquals("added text", Ditto. http://gwt-code-reviews.appspot.com/1340803/diff/1/22#newcode416 user/test/com/google/gwt/user/client/ui/TabLayoutPanelTest.java:416: p.forceLayout(); Why is it necessary to call forceLayout after each layout change? http://gwt-code-reviews.appspot.com/1340803/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
