Store PropertyRegistry::Registrations on CSSInterpolationTypes for registered custom properties

Record PropertyRegistrations on CSSInterpolationTypes for registered custom properties

This change removes the dependency on having access to the Document's
PropertyRegistry during animation value conversion.
This is part of removing the dependency on StyleResolverState when
converting the underlying value to an animation value.
That is part of refactoring CSS Transitions to use InterpolationTypes
instead of AnimatableValues.
That is part of adding support for transitions on registered custom
properties.

This change removes the dependency on Document by storing the
PropertyRegistration on the CSSInterpolationType so that it doesn't
need to look up the registration on Document's PropertyRegistry every
time.

BUG=681424
Review-Url: https://codereview.chromium.org/2654783004
Cr-Commit-Position: refs/heads/master@{#448871}
Committed: https://chromium.googlesource.com/chromium/src/+/b556a4992cb17a53683d83d290c4b4736d1727cd

[alancutter (OOO until 2018), 2017-01-25 07:23:05]

Description was changed from

==========
Store PropertyRegistry::Registrations on CSSInterpolationTypes for registered
custom properties

BUG=681424
==========

to

==========
Store PropertyRegistry::Registrations on CSSInterpolationTypes for registered
custom properties

This change removes the dependency on having access to the Document's
PropertyRegistry during animation value conversion.
This is part of removing the dependency on StyleResolverState when
converting the underlying value to an animation value.
That is part of refactoring CSS Transitions to use InterpolationTypes
instead of AnimatableValues.
That is part of adding support for transitions on registered custom
properties.

BUG=681424
==========

[alancutter (OOO until 2018), 2017-01-25 07:23:06]

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

[Eric Willigers, 2017-01-30 23:35:14]

lgtm

[alancutter (OOO until 2018), 2017-02-07 01:02:36]

Description was changed from

==========
Store PropertyRegistry::Registrations on CSSInterpolationTypes for registered
custom properties

This change removes the dependency on having access to the Document's
PropertyRegistry during animation value conversion.
This is part of removing the dependency on StyleResolverState when
converting the underlying value to an animation value.
That is part of refactoring CSS Transitions to use InterpolationTypes
instead of AnimatableValues.
That is part of adding support for transitions on registered custom
properties.

BUG=681424
==========

to

==========
Record PropertyRegistrations on CSSInterpolationTypes for registered custom
properties

This change removes the dependency on having access to the Document's
PropertyRegistry during animation value conversion.
This is part of removing the dependency on StyleResolverState when
converting the underlying value to an animation value.
That is part of refactoring CSS Transitions to use InterpolationTypes
instead of AnimatableValues.
That is part of adding support for transitions on registered custom
properties.

This change removes the dependency on Document by storing the
PropertyRegistration on the CSSInterpolationType so that it doesn't
need to look up the registration on Document's PropertyRegistry every
time.

BUG=681424
==========

[sashab, 2017-02-07 02:15:08]

Would prefer this moved to PropertyRegistry but the interpolation type is const
on that, so I understand that's not possible. LGTM

[alancutter (OOO until 2018), 2017-02-07 05:06:01]

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

[commit-bot: I haz the power, 2017-02-07 05:06:13]

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, 2017-02-07 06:47:10]

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

[commit-bot: I haz the power, 2017-02-07 06:47:11]

Dry run: This issue passed the CQ dry run.

[alancutter (OOO until 2018), 2017-02-08 02:37:59]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2017-02-08 02:37:59]

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

[commit-bot: I haz the power, 2017-02-08 02:38:23]

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, 2017-02-08 02:43:17]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486521479220030,
"parent_rev": "a0cab13ed385f0c524f29e279b2874677872bcdf", "commit_rev":
"b556a4992cb17a53683d83d290c4b4736d1727cd"}

[commit-bot: I haz the power, 2017-02-08 02:44:18]

Description was changed from

==========
Record PropertyRegistrations on CSSInterpolationTypes for registered custom
properties

This change removes the dependency on having access to the Document's
PropertyRegistry during animation value conversion.
This is part of removing the dependency on StyleResolverState when
converting the underlying value to an animation value.
That is part of refactoring CSS Transitions to use InterpolationTypes
instead of AnimatableValues.
That is part of adding support for transitions on registered custom
properties.

This change removes the dependency on Document by storing the
PropertyRegistration on the CSSInterpolationType so that it doesn't
need to look up the registration on Document's PropertyRegistry every
time.

BUG=681424
==========

to

==========
Record PropertyRegistrations on CSSInterpolationTypes for registered custom
properties

This change removes the dependency on having access to the Document's
PropertyRegistry during animation value conversion.
This is part of removing the dependency on StyleResolverState when
converting the underlying value to an animation value.
That is part of refactoring CSS Transitions to use InterpolationTypes
instead of AnimatableValues.
That is part of adding support for transitions on registered custom
properties.

This change removes the dependency on Document by storing the
PropertyRegistration on the CSSInterpolationType so that it doesn't
need to look up the registration on Document's PropertyRegistry every
time.

BUG=681424

Review-Url: https://codereview.chromium.org/2654783004
Cr-Commit-Position: refs/heads/master@{#448871}
Committed:
https://chromium.googlesource.com/chromium/src/+/b556a4992cb17a53683d83d290c4...
==========

[commit-bot: I haz the power, 2017-02-08 02:44:20]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/b556a4992cb17a53683d83d290c4...