Extend PropertyHandle to include custom property identifiers

Extend PropertyHandle to include custom property identifiers

This change allows animation keyframes to hold custom property
declaration values and treat them as individual properties. This
is achieved by extending PropertyHandles to treat custom properties
based on their string name rather than CSSPropertyID.

This patch has no behavioural change as custom property animations
are not applied yet and therefore not observable. This patch is a
necessary step in supporting custom property animations.

BUG=644148
Committed: https://crrev.com/294c014b805fae756af944846f9ef21a85a71d49
Cr-Commit-Position: refs/heads/master@{#422028}

[alancutter (OOO until 2018), 2016-09-29 09:31:05]

Description was changed from

==========
[WIP] Extend PropertyHandle to include custom property identifiers

BUG=644148
==========

to

==========
Extend PropertyHandle to include custom property identifiers

This change allows animation keyframes to hold custom property
declaration values and treat them as individual properties. This
is achieved by extending PropertyHandles to treat custom properties
based on their string name rather than CSSPropertyID.

This patch has no behavioural change as custom property animations
are not applied yet and therefore not observable. This patch is a
necessary step in supporting custom property animations.


BUG=644148
==========

[alancutter (OOO until 2018), 2016-09-29 09:31:06]

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

[alancutter (OOO until 2018), 2016-09-29 09:36:30]

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

[commit-bot: I haz the power, 2016-09-29 09:37:03]

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-09-29 10:45:39]

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

[commit-bot: I haz the power, 2016-09-29 10:45:40]

Dry run: This issue passed the CQ dry run.

[Eric Willigers, 2016-09-29 22:34:44]

lgtm

[alancutter (OOO until 2018), 2016-09-30 00:19:34]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-09-30 00:20:07]

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-09-30 00:30:16]

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

[commit-bot: I haz the power, 2016-09-30 00:30:17]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[alancutter (OOO until 2018), 2016-09-30 00:32:15]

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

[alancutter (OOO until 2018), 2016-09-30 00:32:15]

+timloh for StyleResolver comment review.

[Timothy Loh, 2016-09-30 04:17:27]

On 2016/09/30 00:32:15, alancutter wrote:
> +timloh for StyleResolver comment review.

stamp lgtm

[alancutter (OOO until 2018), 2016-09-30 04:27:19]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-09-30 04:27:37]

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-09-30 04:33:20]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-30 04:37:55]

Description was changed from

==========
Extend PropertyHandle to include custom property identifiers

This change allows animation keyframes to hold custom property
declaration values and treat them as individual properties. This
is achieved by extending PropertyHandles to treat custom properties
based on their string name rather than CSSPropertyID.

This patch has no behavioural change as custom property animations
are not applied yet and therefore not observable. This patch is a
necessary step in supporting custom property animations.


BUG=644148
==========

to

==========
Extend PropertyHandle to include custom property identifiers

This change allows animation keyframes to hold custom property
declaration values and treat them as individual properties. This
is achieved by extending PropertyHandles to treat custom properties
based on their string name rather than CSSPropertyID.

This patch has no behavioural change as custom property animations
are not applied yet and therefore not observable. This patch is a
necessary step in supporting custom property animations.

BUG=644148

Committed: https://crrev.com/294c014b805fae756af944846f9ef21a85a71d49
Cr-Commit-Position: refs/heads/master@{#422028}
==========

[commit-bot: I haz the power, 2016-09-30 04:37:56]

Patchset 5 (id:??) landed as
https://crrev.com/294c014b805fae756af944846f9ef21a85a71d49
Cr-Commit-Position: refs/heads/master@{#422028}