LGTM
https://codereview.appspot.com/30570043/diff/1/src/com/google/caja/plugin/sanitizecss.js
File src/com/google/caja/plugin/sanitizecss.js (right):
https://codereview.appspot.com/30570043/diff/1/src/com/google/caja/plugin/sanitizecss.js#newcode643
src/com/google/caja/plugin/sanitizecss.js:643:
HISTORY_NON_SENSITIVE_PSEUDO_SELECTOR_WHITELIST.test(tok)) {
fix while we're at it: continuation line should be four-space indented
https://codereview.appspot.com/30570043/diff/1/src/com/google/caja/plugin/sanitizecss.js#newcode644
src/com/google/caja/plugin/sanitizecss.js:644: historySensitive =
historySensitive || false;
This doesn't seem worth keeping around. If you want to point out that
we're not setting it to true in this branch, maybe just a comment?
https://codereview.appspot.com/30570043/
--
---
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.
