Reviewers: skybrian,

Description:
Thanks minichate!
GWT CSS parser was dropping media types.

Fixes issue: 4911


Please review this at http://gwt-code-reviews.appspot.com/1863803/

Affected files:
  M user/src/com/google/gwt/resources/css/CssGenerationVisitor.java
  M user/src/com/google/gwt/resources/css/ast/CssNodeCloner.java


Index: user/src/com/google/gwt/resources/css/CssGenerationVisitor.java
===================================================================
--- user/src/com/google/gwt/resources/css/CssGenerationVisitor.java (revision 11354) +++ user/src/com/google/gwt/resources/css/CssGenerationVisitor.java (working copy)
@@ -185,7 +185,7 @@

   @Override
   public boolean visit(CssMediaRule x, Context ctx) {
-    out.print("@MEDIA");
+    out.print("@media");
     for (String m : x.getMedias()) {
       out.print(" " + m);
     }
Index: user/src/com/google/gwt/resources/css/ast/CssNodeCloner.java
===================================================================
--- user/src/com/google/gwt/resources/css/ast/CssNodeCloner.java (revision 11354) +++ user/src/com/google/gwt/resources/css/ast/CssNodeCloner.java (working copy)
@@ -180,7 +180,7 @@
   @Override
   public boolean visit(CssMediaRule x, Context ctx) {
     CssMediaRule newRule = new CssMediaRule();
-    newRule.getMedias().addAll(newRule.getMedias());
+    newRule.getMedias().addAll(x.getMedias());

     pushNodes(newRule);
     return true;


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

Reply via email to