Reviewers: zundel, Description: Re-enable a few style tests now that old linux hosted mode is gone
Please review this at http://gwt-code-reviews.appspot.com/1228801/show Affected files: M user/test/com/google/gwt/dom/client/StyleTest.java Index: user/test/com/google/gwt/dom/client/StyleTest.java =================================================================== --- user/test/com/google/gwt/dom/client/StyleTest.java (revision 9425) +++ user/test/com/google/gwt/dom/client/StyleTest.java (working copy) @@ -85,14 +85,13 @@ style.setCursor(Cursor.HELP); assertEquals(Cursor.HELP, style.getCursor()); - // These aren't supported on old mozilla, so testing them will break. - // TODO: re-enable these cases when we finally drop linux hosted mode. - /* + // Note, this test will fail on old mozilla due to unsupported style style.setCursor(Cursor.COL_RESIZE); assertEquals(Cursor.COL_RESIZE, style.getCursor()); + + // Note, this test will fail on old mozilla due to unsupported style style.setCursor(Cursor.ROW_RESIZE); assertEquals(Cursor.ROW_RESIZE, style.getCursor()); - */ } public void testDisplay() { @@ -106,12 +105,9 @@ style.setDisplay(Display.INLINE); assertEquals(Display.INLINE, style.getDisplay()); - // Not supported on old mozilla, so testing it will break. - // TODO: re-enable these cases when we finally drop linux hosted mode. - /* + // Note, this test will fail on old mozilla due to unsupported style style.setDisplay(Display.INLINE_BLOCK); assertEquals(Display.INLINE_BLOCK, style.getDisplay()); - */ } public void testFontStyle() { -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
