LGTM
https://codereview.appspot.com/13683045/diff/1/src/com/google/caja/plugin/CssRewriter.java
File src/com/google/caja/plugin/CssRewriter.java (right):
https://codereview.appspot.com/13683045/diff/1/src/com/google/caja/plugin/CssRewriter.java#newcode603
src/com/google/caja/plugin/CssRewriter.java:603: private static final
TypesafeSet<Name> ALLOWED_PSEUDO_CLASSES =
Suggest citing the reference this list was constructed from.
https://codereview.appspot.com/13683045/diff/1/src/com/google/caja/plugin/sanitizecss.js
File src/com/google/caja/plugin/sanitizecss.js (right):
https://codereview.appspot.com/13683045/diff/1/src/com/google/caja/plugin/sanitizecss.js#newcode366
src/com/google/caja/plugin/sanitizecss.js:366: // See also
ALLOWED_PSEUDO_CLASSES in CssRewriter.java
Ditto. Also explicitly say it's duplicated rather than just "see also".
Since the Java code is hopefully going away in the future, this JS code
should have at least as much commentary as the Java code.
https://codereview.appspot.com/13683045/
--
---
You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
