lgtm.
you might want to add a testcase in
tests/.../plugin/css-stylesheet-tests.js
https://codereview.appspot.com/85040043/diff/1/src/com/google/caja/lang/html/html4-attributes-defs.json
File src/com/google/caja/lang/html/html4-attributes-defs.json (right):
https://codereview.appspot.com/85040043/diff/1/src/com/google/caja/lang/html/html4-attributes-defs.json#newcode563
src/com/google/caja/lang/html/html4-attributes-defs.json:563: "comment":
"-1 and on-any-element are extensions/HTML5" },
The historic pattern was to express these extensions as override rules
in html5-elements-defs, but I'm fine with it here, because the
html4/html5 distinction is not very useful, there are no cases where
someone wants to permit strict html4 excluding html5, and I had a TODO
to collapse them together anyway.
https://codereview.appspot.com/85040043/
--
---
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/d/optout.
