lgtm
https://codereview.appspot.com/12837046/diff/1/src/com/google/caja/plugin/CssValidator.java
File src/com/google/caja/plugin/CssValidator.java (right):
https://codereview.appspot.com/12837046/diff/1/src/com/google/caja/plugin/CssValidator.java#newcode1347
src/com/google/caja/plugin/CssValidator.java:1347: // Unrecognized atom:
fail the match.
I guess the checks you handled to CssPropertyPatterns will still help us
suss out problems like this statically in the main branch and this
degrades more gracefully
https://codereview.appspot.com/12837046/diff/1/tests/com/google/caja/plugin/CssValidatorTest.java
File tests/com/google/caja/plugin/CssValidatorTest.java (right):
https://codereview.appspot.com/12837046/diff/1/tests/com/google/caja/plugin/CssValidatorTest.java#newcode50
tests/com/google/caja/plugin/CssValidatorTest.java:50: "WARNING: css
property animation-name has bad value: ==>foo<==");
Ok. Maybe a comment that foo could be ok, but foo__ should not be.
https://codereview.appspot.com/12837046/
--
---
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.
