GSS Conversion bug with several undefined constants. The converter fails to convert everything in lower case when a property value contains several uppercase strings that don't match constants. Bug: issue 8963 Change-Id: Ief059cb704bd312dd32508d1ada853874c9c7832 (cherry picked from commit 551d490fb98831848f2d2bc7a1e81ac59ef48ff5)
diff --git a/user/src/com/google/gwt/resources/converter/UndefinedConstantVisitor.java b/user/src/com/google/gwt/resources/converter/UndefinedConstantVisitor.java index 956d301..d37d037 100644 --- a/user/src/com/google/gwt/resources/converter/UndefinedConstantVisitor.java +++ b/user/src/com/google/gwt/resources/converter/UndefinedConstantVisitor.java
@@ -79,7 +79,6 @@ + cssProperty.getName() + "' from rule '" + x.getSelectors().toString() + "' undefined constant: " + upperCaseString); } - break; } }
diff --git a/user/test/com/google/gwt/resources/converter/undefined-constants.css b/user/test/com/google/gwt/resources/converter/undefined-constants.css index d6b263d..55ae751 100644 --- a/user/test/com/google/gwt/resources/converter/undefined-constants.css +++ b/user/test/com/google/gwt/resources/converter/undefined-constants.css
@@ -3,3 +3,7 @@ color: JFK_MY_STUFF; background-color: JFK_123_ASD; } + +.cellTableHeader { + padding: HEADER_VERTICAL_PADDING HEADER_HORIZONTAL_PADDING; +}
diff --git a/user/test/com/google/gwt/resources/converter/undefined-constants.gss b/user/test/com/google/gwt/resources/converter/undefined-constants.gss index 0813069..9f977a9 100644 --- a/user/test/com/google/gwt/resources/converter/undefined-constants.gss +++ b/user/test/com/google/gwt/resources/converter/undefined-constants.gss
@@ -4,3 +4,7 @@ color: jfk_my_stuff; background-color: jfk_123_asd; } + +.cellTableHeader { + padding: header_vertical_padding header_horizontal_padding; +}