Reviewers: Jasvir,

Description:
sanitizerSelectors(cssText) performs the same sanitization as
CssRewriter.  This CL is just a testfile that contains both the code
under test and the testcases in CssRewriterTest and does not attempt
to move the code into it's final location since I want to discuss
where this should fit as part of this CL.

One issue is whether CssRewriterTest.java should run its tests against
the
JavaScript version.  To do that, we would need to

1. either load html-emitter.js and domado.js into Rhino to get the
property
sanitization function in domado.js, OR

2. move the property sanitization out of domado.js so that less needs
to be pulled into Rhino to test equivalence of the java and javascript
CSS rewritings.

I will follow this CL shortly with another that wires
sanitizeSelectors into html-emitter.js.

Please review this at http://codereview.appspot.com/5649058/

Affected files:
  M     src/com/google/caja/plugin/cssparser.js
  MM    src/com/google/caja/plugin/domado.js
  M     src/com/google/caja/plugin/html-emitter.js
  M     tests/com/google/caja/plugin/CssRewriterTest.java
  A     tests/com/google/caja/plugin/css-selector-test.html
  A     tests/com/google/caja/plugin/css-selector-test.js


Reply via email to