Make UseCounters work on aliased properties

Make UseCounters work on aliased properties

This patch starts to build a coherent system for aliased properties.
The general plan is for property aliases to get resolved as early as
possible. Properties with no differences or minor differences in parsing
will use this code-path, while properties with larger differences will
still require aliases to leak into the rest of the system.

Property aliases are generally resolved during parsing (although for
will-change and transition-property we leave aliases in), so that we
only store resolved properties on StylePropertySets. Property aliases
which parse slightly differently will parse into CSSValues which are
also valid for the aliased properties.

The handling here is similar to shorthands, where we don't have the type
safety of a separate enum, but functions that don't take aliases will
simply assume that they aren't given aliases. This is as there is no
nice way to do enum inheritance in C++.

Eventually most aliases will use the new system. For now, we only move
across properties which are already marked as alias_for, making the
UseCounters work for these.

BUG=277126, 410112
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194021

[Timothy Loh, 2015-04-20 01:03:53]

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

[alancutter (OOO until 2018), 2015-04-20 06:19:37]

lgtm

https://codereview.chromium.org/1096963002/diff/40001/Source/build/scripts/cs...
File Source/build/scripts/css_properties.py (right):

https://codereview.chromium.org/1096963002/diff/40001/Source/build/scripts/cs...
Source/build/scripts/css_properties.py:55: # StylePropertyMetadata also assumes
there are under 1024 properties.
Why not just assert this?

[Timothy Loh, 2015-04-20 07:14:29]

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

[Timothy Loh, 2015-04-20 07:14:30]

+haraken for Source/bindings OWNERS

https://codereview.chromium.org/1096963002/diff/40001/Source/build/scripts/cs...
File Source/build/scripts/css_properties.py (right):

https://codereview.chromium.org/1096963002/diff/40001/Source/build/scripts/cs...
Source/build/scripts/css_properties.py:55: # StylePropertyMetadata also assumes
there are under 1024 properties.
On 2015/04/20 06:19:37, alancutter wrote:
> Why not just assert this?

I'm pretty sure this will read really weirdly if I make this an assertions. I
updated the comment though to be a bit clearer.

[haraken, 2015-04-20 07:20:21]

bindings/ LGTM

[Timothy Loh, 2015-04-20 07:21:26]

The CQ bit was checked by timloh@chromium.org

[Timothy Loh, 2015-04-20 07:21:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org
Link to the patchset: https://codereview.chromium.org/1096963002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-04-20 07:21:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1096963002/60001

[commit-bot: I haz the power, 2015-04-20 12:03:22]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194021