On 2013/07/27 02:19:21, MikeSamuel wrote:
https://codereview.appspot.com/11949043/diff/5001/tests/com/google/caja/plugin/sanitizecss_test.js
File tests/com/google/caja/plugin/sanitizecss_test.js (right):
https://codereview.appspot.com/11949043/diff/5001/tests/com/google/caja/plugin/sanitizecss_test.js#newcode579
tests/com/google/caja/plugin/sanitizecss_test.js:579:
'transition-property:opacity , , left;', // bogus elided
On 2013/07/26 23:35:27, felix8a wrote:
> eliding properties makes the value invalid, there needs to be at
least a
> placeholder item. the spec doesn't define any allowed null value,
the keyword
> 'none' is invalid in this context. but if we really want to filter
invalid
> props, we can use some value like 'invalid'.
We're doing token filtering now. We could drop the property entirely,
or have a
per-property substitution token, but that's a fairly significant
change.
ok, lgtm. if the elision becomes a problem we can fix that later.
https://codereview.appspot.com/11949043/
--
---
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.
