Preliminary support for CSS3 animations

This CL:

1. Adds support to sanitizeStylesheet for @keyframes rules
2. Adds a CSS property bit for the <global-name> CSS value type
3. Adds entries to the CSS schemas for animation and animation-* properties
4. Adds partial support for <global-nam>s to sanitizeCssProperty
5. Adds tests for well-formed and malformed @keyframes rules

Still TODO: thread the ID suffix to sanitizeCssProperty for use in
mangling global names.  Possibly pass the same virtualization bundle
to sanitizeCssProperty as is passed to other sanitizeCss* functions.

----

Submitted @r5510

[MikeSamuel, 2013-07-25 14:19:21]

This CL:

1. Adds support to sanitizeStylesheet for @keyframes rules
2. Adds a CSS property bit for the <global-name> CSS value type
3. Adds entries to the CSS schemas for animation and animation-* properties
4. Adds partial support for <global-nam>s to sanitizeCssProperty
5. Adds tests for well-formed and malformed @keyframes rules

Still TODO: thread the ID suffix to sanitizeCssProperty for use in
mangling global names.  Possibly pass the same virtualization bundle
to sanitizeCssProperty as is passed to other sanitizeCss* functions.

[felix8a, 2013-07-25 18:01:14]

lgtm

https://codereview.appspot.com/11832043/diff/3001/src/com/google/caja/plugin/...
File src/com/google/caja/plugin/sanitizecss.js (right):

https://codereview.appspot.com/11832043/diff/3001/src/com/google/caja/plugin/...
src/com/google/caja/plugin/sanitizecss.js:24: * \@requires
CSS_PROP_BIT_GLOBAL_NAME
move before HASH_VALUE for alphabetical order

https://codereview.appspot.com/11832043/diff/3001/src/com/google/caja/plugin/...
src/com/google/caja/plugin/sanitizecss.js:812: &&
!/__$|[^#0-9A-Za-z:_\-]/.test(animationId)) {
this regex allows ':' and '#' but the GLOBAL_NAME regex above doesn't. should
they be the same?

[MikeSamuel, 2013-07-25 21:33:46]

This CL:

1. Adds support to sanitizeStylesheet for @keyframes rules
2. Adds a CSS property bit for the <global-name> CSS value type
3. Adds entries to the CSS schemas for animation and animation-* properties
4. Adds partial support for <global-nam>s to sanitizeCssProperty
5. Adds tests for well-formed and malformed @keyframes rules

Still TODO: thread the ID suffix to sanitizeCssProperty for use in
mangling global names.  Possibly pass the same virtualization bundle
to sanitizeCssProperty as is passed to other sanitizeCss* functions.

[MikeSamuel, 2013-07-25 21:33:51]

https://codereview.appspot.com/11832043/diff/3001/src/com/google/caja/plugin/...
File src/com/google/caja/plugin/sanitizecss.js (right):

https://codereview.appspot.com/11832043/diff/3001/src/com/google/caja/plugin/...
src/com/google/caja/plugin/sanitizecss.js:24: * \@requires
CSS_PROP_BIT_GLOBAL_NAME
On 2013/07/25 18:01:15, felix8a wrote:
> move before HASH_VALUE for alphabetical order

Done.

https://codereview.appspot.com/11832043/diff/3001/src/com/google/caja/plugin/...
src/com/google/caja/plugin/sanitizecss.js:812: &&
!/__$|[^#0-9A-Za-z:_\-]/.test(animationId)) {
On 2013/07/25 18:01:15, felix8a wrote:
> this regex allows ':' and '#' but the GLOBAL_NAME regex above doesn't. should
> they be the same?

Good catch.

I'm not sure what to do since ":" would be a separate token.  I could merge the
headerArray on empty string to get the animationId, but doing that in the
property name is going to complicate things, so I dropped the : and # from here.

[felix8a, 2013-07-25 22:05:27]

lgtm, with fix below

https://codereview.appspot.com/11832043/diff/14001/src/com/google/caja/plugin...
File src/com/google/caja/plugin/sanitizecss.js (right):

https://codereview.appspot.com/11832043/diff/14001/src/com/google/caja/plugin...
src/com/google/caja/plugin/sanitizecss.js:812: &&
!/__$|[^#0-9A-Za-z:_\-]/.test(animationId)) {
need to change this regexp too. (you changed one above but not this one)

[MikeSamuel, 2013-07-26 00:03:35]

This CL:

1. Adds support to sanitizeStylesheet for @keyframes rules
2. Adds a CSS property bit for the <global-name> CSS value type
3. Adds entries to the CSS schemas for animation and animation-* properties
4. Adds partial support for <global-nam>s to sanitizeCssProperty
5. Adds tests for well-formed and malformed @keyframes rules

Still TODO: thread the ID suffix to sanitizeCssProperty for use in
mangling global names.  Possibly pass the same virtualization bundle
to sanitizeCssProperty as is passed to other sanitizeCss* functions.

[MikeSamuel, 2013-07-26 00:04:23]

https://codereview.appspot.com/11832043/diff/14001/src/com/google/caja/plugin...
File src/com/google/caja/plugin/sanitizecss.js (right):

https://codereview.appspot.com/11832043/diff/14001/src/com/google/caja/plugin...
src/com/google/caja/plugin/sanitizecss.js:812: &&
!/__$|[^#0-9A-Za-z:_\-]/.test(animationId)) {
On 2013/07/25 22:05:27, felix8a wrote:
> need to change this regexp too. (you changed one above but not this one)

Done.

[felix8a, 2013-07-26 00:14:11]

lgtm