LGTM Thanks for this fix. Can you fill out a CLA so we can use the patch? If you scroll to the bottom of the link below, you can fill it out online: http://code.google.com/legal/individual-cla-v1.0.html
http://gwt-code-reviews.appspot.com/1353801/diff/1/user/test/com/google/gwt/dom/client/FrameTests.java File user/test/com/google/gwt/dom/client/FrameTests.java (right): http://gwt-code-reviews.appspot.com/1353801/diff/1/user/test/com/google/gwt/dom/client/FrameTests.java#newcode44 user/test/com/google/gwt/dom/client/FrameTests.java:44: int delayMillis = 3000; You can abstract this to a static final variable: private static final int FRAME_LOAD_DELAY = 3000; http://gwt-code-reviews.appspot.com/1353801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
