Reviewers: Ray Ryan, kjin,

Message:
Kevin, thanks for looking at this.  Are there any other CSS testing
use-cases that you can think of that the DebugInfo API should support?


http://gwt-code-reviews.appspot.com/770801/diff/1/5
File
user/src/com/google/gwt/resources/client/impl/CssResourceObserver.java
(right):

http://gwt-code-reviews.appspot.com/770801/diff/1/5#newcode32
user/src/com/google/gwt/resources/client/impl/CssResourceObserver.java:32:
private static final CssResourceObserver IMPL =
GWT.create(CssResourceObserver.class);
This way there's effectively only one observer instance allowed?
I think Observer implies 1-N.

It's a singleton observer the way it's written, but if the developer
really cares, a fan-out version could be installed.

http://gwt-code-reviews.appspot.com/770801/diff/1/5#newcode68
user/src/com/google/gwt/resources/client/impl/CssResourceObserver.java:68:
private native JavaScriptObject ensureMap() /*-{
Do we have to use JSNI?

Yes.  The couple of requests that I've seen for the global map have a
use-case where a white-box Selenium test would like to be able to
determine the obfuscated name for a class selector when inspecting the
DOM.

http://gwt-code-reviews.appspot.com/770801/diff/1/6
File user/src/com/google/gwt/resources/css/ClassRenamer.java (right):

http://gwt-code-reviews.appspot.com/770801/diff/1/6#newcode138
user/src/com/google/gwt/resources/css/ClassRenamer.java:138:
sheet.getDebugInfo().addToClassMap(sourceClassName, sourceClassName);
It is a builder and should probably be named something like
DebugInfoModel.

Description:
Add debugging information to CssResource.
Patch by: bobv
Review by: rjrjr


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

Affected files:
  A user/src/com/google/gwt/resources/EnableCssResourceDebugging.gwt.xml
  M user/src/com/google/gwt/resources/Resources.gwt.xml
  M user/src/com/google/gwt/resources/client/CssResource.java
  A user/src/com/google/gwt/resources/client/impl/CssResourceObserver.java
  M user/src/com/google/gwt/resources/css/ClassRenamer.java
  A user/src/com/google/gwt/resources/css/CssDebugInfo.java
  A user/src/com/google/gwt/resources/css/CssDebugInfoImpl.java
  M user/src/com/google/gwt/resources/css/ast/CssStylesheet.java
  M user/src/com/google/gwt/resources/rg/CssResourceGenerator.java
  M user/test/com/google/gwt/resources/ResourcesSuite.java
  M user/test/com/google/gwt/resources/client/CSSResourceTest.java
  A user/test/com/google/gwt/resources/client/CssResourceDebugInfoTest.java


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

Reply via email to