Disallow CSS custom properties as descriptors

Disallow CSS custom properties as descriptors

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873
Committed: https://crrev.com/bfc0527d0b204f20d238d8e998f9485b637dd4a3
Cr-Commit-Position: refs/heads/master@{#430484}

[Timothy Loh, 2016-11-04 03:36:27]

The CQ bit was checked by timloh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-04 03:37:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-04 04:10:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-04 04:10:53]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Timothy Loh, 2016-11-07 02:08:12]

Description was changed from

==========
---------------------This line is 72 characters long--------------------
do something i'm not really sure what because the branch is a couple weeks old
==========

to

==========
Disallow CSS custom properties in at-rules

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873
==========

[Timothy Loh, 2016-11-07 02:08:12]

timloh@chromium.org changed reviewers:
+ alancutter@chromium.org

[Timothy Loh, 2016-11-07 02:10:39]

Description was changed from

==========
Disallow CSS custom properties in at-rules

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873
==========

to

==========
Disallow CSS custom properties as descriptors

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873
==========

[rune, 2016-11-07 09:48:39]

rune@opera.com changed reviewers:
+ rune@opera.com

[rune, 2016-11-07 09:48:39]

lgtm

[Timothy Loh, 2016-11-08 00:26:14]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2016-11-08 00:26:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-08 02:34:48]

Description was changed from

==========
Disallow CSS custom properties as descriptors

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873
==========

to

==========
Disallow CSS custom properties as descriptors

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873
==========

[commit-bot: I haz the power, 2016-11-08 02:34:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 03:27:21]

Description was changed from

==========
Disallow CSS custom properties as descriptors

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873
==========

to

==========
Disallow CSS custom properties as descriptors

CSS custom properties are properties so should be not allowed where we
only otherwise allow descriptors. This patch makes the parser reject
custom properties in at rules (excluding in keyframe blocks, which take
properties instead of descriptors).

BUG=641873

Committed: https://crrev.com/bfc0527d0b204f20d238d8e998f9485b637dd4a3
Cr-Commit-Position: refs/heads/master@{#430484}
==========

[commit-bot: I haz the power, 2016-11-08 03:27:22]

Patchset 1 (id:??) landed as
https://crrev.com/bfc0527d0b204f20d238d8e998f9485b637dd4a3
Cr-Commit-Position: refs/heads/master@{#430484}