Recognize media queries on @import, <link media=...>, and <style media=...>

This changes html-emitter and sanitizeStylesheet to recognize media queries,
and sanitize them appropriately.

Previously, there was a bug that caused far too many stylesheets to be
generated -- the style element would have multiple text nodes, each a
prefix of later ones.  This CL inlines @imports where they occur in
the stylesheet.

Tests for corner cases like nested media
  <style media="only all and (color)">
  @import "bar.css" only all and (hover)
  </style>
are in es53-test-css-imports

----

Submitted @ r5517

[MikeSamuel, 2013-07-31 05:09:27]

This changes html-emitter and sanitizeStylesheet to recognize media queries,
and sanitize them appropriately.

Previously, there was a bug that caused far too many stylesheets to be
generated -- the style element would have multiple text nodes, each a
prefix of later ones.  This CL inlines @imports where they occur in
the stylesheet.

STILL PENDING: tests for corner cases like nested media
  <style media="only all and (color)">
  @import "bar.css" only all and (hover)
  </style>
Should these go in es53-test-css-imports

[MikeSamuel, 2013-07-31 07:18:37]

This changes html-emitter and sanitizeStylesheet to recognize media queries,
and sanitize them appropriately.

Previously, there was a bug that caused far too many stylesheets to be
generated -- the style element would have multiple text nodes, each a
prefix of later ones.  This CL inlines @imports where they occur in
the stylesheet.

STILL PENDING: tests for corner cases like nested media
  <style media="only all and (color)">
  @import "bar.css" only all and (hover)
  </style>
Should these go in es53-test-css-imports

[felix8a, 2013-07-31 19:18:25]

On 2013/07/31 07:18:37, MikeSamuel wrote:
> STILL PENDING: tests for corner cases like nested media
>   <style media="only all and (color)">
>   @import "bar.css" only all and (hover)
>   </style>
> Should these go in es53-test-css-imports

yeah, es53-test-css-imports seems like an ok place to test them.

lgtm so far

[MikeSamuel, 2013-07-31 23:58:45]

This changes html-emitter and sanitizeStylesheet to recognize media queries,
and sanitize them appropriately.

Previously, there was a bug that caused far too many stylesheets to be
generated -- the style element would have multiple text nodes, each a
prefix of later ones.  This CL inlines @imports where they occur in
the stylesheet.

STILL PENDING: tests for corner cases like nested media
  <style media="only all and (color)">
  @import "bar.css" only all and (hover)
  </style>
Should these go in es53-test-css-imports

[MikeSamuel, 2013-07-31 23:59:29]

This changes html-emitter and sanitizeStylesheet to recognize media queries,
and sanitize them appropriately.

Previously, there was a bug that caused far too many stylesheets to be
generated -- the style element would have multiple text nodes, each a
prefix of later ones.  This CL inlines @imports where they occur in
the stylesheet.

Tests for corner cases like nested media
  <style media="only all and (color)">
  @import "bar.css" only all and (hover)
  </style>
are in es53-test-css-imports

[MikeSamuel, 2013-08-01 00:00:26]

On 2013/07/31 19:18:25, felix8a wrote:
> yeah, es53-test-css-imports seems like an ok place to test them.

Done.

[MikeSamuel, 2013-08-01 15:40:13]

This changes html-emitter and sanitizeStylesheet to recognize media queries,
and sanitize them appropriately.

Previously, there was a bug that caused far too many stylesheets to be
generated -- the style element would have multiple text nodes, each a
prefix of later ones.  This CL inlines @imports where they occur in
the stylesheet.

Tests for corner cases like nested media
  <style media="only all and (color)">
  @import "bar.css" only all and (hover)
  </style>
are in es53-test-css-imports

[MikeSamuel, 2013-08-01 15:40:43]

snapshotted

[felix8a, 2013-08-02 05:03:49]

lgtm

https://codereview.appspot.com/12138043/diff/17001/tests/com/google/caja/plug...
File tests/com/google/caja/plugin/es53-test-css-imports-guest.html (right):

https://codereview.appspot.com/12138043/diff/17001/tests/com/google/caja/plug...
tests/com/google/caja/plugin/es53-test-css-imports-guest.html:34: media="all and
(color) and (color: 0)">
maybe omit "all and" to verify that implied all works. most html/css in the wild
will probably omit "all and".

[MikeSamuel, 2013-08-02 06:20:18]

On 2013/08/02 05:03:49, felix8a wrote:
> lgtm
> 
>
https://codereview.appspot.com/12138043/diff/17001/tests/com/google/caja/plug...
> File tests/com/google/caja/plugin/es53-test-css-imports-guest.html (right):
> 
>
https://codereview.appspot.com/12138043/diff/17001/tests/com/google/caja/plug...
> tests/com/google/caja/plugin/es53-test-css-imports-guest.html:34: media="all
and
> (color) and (color: 0)">
> maybe omit "all and" to verify that implied all works. most html/css in the
wild
> will probably omit "all and".

Done