@r5636
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)) {
On 2013/11/21 23:36:15, kpreid2 wrote:
fix while we're at it: continuation line should be four-space indented
Done.
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;
On 2013/11/21 23:36:15, kpreid2 wrote:
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?
yeah I noticed that and deleted it slightly before you submitted review,
heh.
I'm thinking of deleting all the historysensitive code in a separate CL,
since it's unnecessary in all the browsers we support
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.
