http://gwt-code-reviews.appspot.com/1735804/diff/1/user/src/com/google/gwt/resources/css/SubstitutionReplacer.java File user/src/com/google/gwt/resources/css/SubstitutionReplacer.java (right):
http://gwt-code-reviews.appspot.com/1735804/diff/1/user/src/com/google/gwt/resources/css/SubstitutionReplacer.java#newcode71 user/src/com/google/gwt/resources/css/SubstitutionReplacer.java:71: for (ListIterator<Value> i = values.listIterator(); i.hasNext();) { Some cleanup that's unrelated to your change (so you might not want to do it), but will make this code easier to understand: I think we shouldn't modify the list in place, but instead append all the values to a new list. Something like this: List<Value> result = new ArrayList<Value>(original.size()); for (Value val : original) { ... append stuff to result ... } return new ListValue(result); http://gwt-code-reviews.appspot.com/1735804/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
