commit 6e4bd8ff2235a6d6d6c5520f31d58097dc17ba2d
author fredsa@google.com <fredsa@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Wed Jan 05 22:37:41 2011 +0000
committer fredsa@google.com <fredsa@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Wed Jan 05 22:37:41 2011 +0000
tree c0458c6a104b5353099aa4b065081347b16479e7
parent f30ddd46ce44ca3bfd5a7ce1393ba44b37efa5db

Re-enable a few style tests now that old linux hosted mode is gone

Review at http://gwt-code-reviews.appspot.com/1228801

Review by: zundel@google.com

git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@9499 8db76d5a-ed1c-0410-87a9-c151d255dfc7

user/test/com/google/gwt/dom/client/StyleTest.java

diff --git a/user/test/com/google/gwt/dom/client/StyleTest.java b/user/test/com/google/gwt/dom/client/StyleTest.java
index b8842ba..9e17047 100644
--- a/user/test/com/google/gwt/dom/client/StyleTest.java
+++ b/user/test/com/google/gwt/dom/client/StyleTest.java
@@ -36,6 +36,8 @@
 import com.google.gwt.dom.client.Style.TextDecoration;
 import com.google.gwt.dom.client.Style.VerticalAlign;
 import com.google.gwt.dom.client.Style.Visibility;
+import com.google.gwt.junit.DoNotRunWith;
+import com.google.gwt.junit.Platform;
 import com.google.gwt.junit.client.GWTTestCase;
 
 /**
@@ -85,14 +87,19 @@
     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 will test fail on old mozilla (prior to gecko 1.8) due to
+     * unsupported style property
+     */
     style.setCursor(Cursor.COL_RESIZE);
     assertEquals(Cursor.COL_RESIZE, style.getCursor());
+
+    /*
+     * Note, this will test fail on old mozilla (prior to gecko 1.8) due to
+     * unsupported style property
+     */
     style.setCursor(Cursor.ROW_RESIZE);
     assertEquals(Cursor.ROW_RESIZE, style.getCursor());
-    */
   }
 
   public void testDisplay() {
@@ -106,12 +113,12 @@
     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 will test fail on old mozilla (prior to gecko 1.8) due to
+     * unsupported style property
+     */
     style.setDisplay(Display.INLINE_BLOCK);
     assertEquals(Display.INLINE_BLOCK, style.getDisplay());
-    */
   }
 
   public void testFontStyle() {
@@ -241,6 +248,7 @@
     assertEquals(Visibility.HIDDEN, style.getVisibility());
   }
 
+  @DoNotRunWith({Platform.HtmlUnitUnknown})
   public void testUnits() {
     DivElement div = Document.get().createDivElement();
     Style style = div.getStyle();