Discussed in person with jat and scottb: Tests are failing for this change because the ThreadLocal assumptions don't actually hold under JUnit. As a result, I'm going to be dropping this change in favor of more explicitly caching based on the generator context lifecycle.
On 2010/08/20 20:46:05, jat wrote:
LGTM, though your format settings still seem to be off.
http://gwt-code-reviews.appspot.com/766803/diff/29001/30006 File
user/src/com/google/gwt/resources/rebind/context/AbstractResourceContext.java
(right):
http://gwt-code-reviews.appspot.com/766803/diff/29001/30006#newcode87
user/src/com/google/gwt/resources/rebind/context/AbstractResourceContext.java:87:
getLogger().log(TreeLogger.ERROR, "Unable to determine mime type of
resource",
e); Formatting changes still seem wrong here, as this now exceeds 80
columns. http://gwt-code-reviews.appspot.com/766803/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors