Store registered property InterpolationTypes in PropertyRegistration

Store registered property InterpolationTypes in PropertyRegistration

This change moves the storage of InterpolationTypes for a registered
custom property to its PropertyRegistration object.
This is in preparation for supporting smooth interpolations on
registered custom properties.

There are no behavioural changes made by this patch.

BUG=671904
Review-Url: https://codereview.chromium.org/2618673003
Cr-Commit-Position: refs/heads/master@{#442204}
Committed: https://chromium.googlesource.com/chromium/src/+/3e5152f0ec19bdbc37aa171be11690f86a6ab311

[alancutter (OOO until 2018), 2017-01-05 04:45:12]

Description was changed from

==========
Store registered property InterpolationTypes in PropertyRegistration

This change moves the storage of InterpolationTypes for a registered
custom property to its PropertyRegistration object.
This is in preparation for supporting smooth interpolations on
registered custom properties.

There are no behavioural changes made by this patch.

BUG=
==========

to

==========
Store registered property InterpolationTypes in PropertyRegistration

This change moves the storage of InterpolationTypes for a registered
custom property to its PropertyRegistration object.
This is in preparation for supporting smooth interpolations on
registered custom properties.

There are no behavioural changes made by this patch.

BUG=671904
==========

[alancutter (OOO until 2018), 2017-01-05 04:45:12]

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

[Timothy Loh, 2017-01-09 05:07:23]

lgtm but I'm not really familiar with interpolation stuff.

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/PropertyRegistration.cpp (right):

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/PropertyRegistration.cpp:7: #include
"core/animation/CSSColorInterpolationType.h"
not used yet

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/Document.h (right):

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/Document.h:1299: const PropertyRegistry*
propertyRegistry() const;
needs a rebase, this exists now :O

[alancutter (OOO until 2018), 2017-01-09 05:32:15]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2017-01-09 05:32:17]

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

[commit-bot: I haz the power, 2017-01-09 05:32:24]

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-01-09 05:42:57]

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

[commit-bot: I haz the power, 2017-01-09 05:42:58]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[alancutter (OOO until 2018), 2017-01-09 05:45:14]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2017-01-09 05:45:14]

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

[commit-bot: I haz the power, 2017-01-09 05:45:27]

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-01-09 07:14:32]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483940714279440,
"parent_rev": "854380e3ef0dc2d5cfdc171bb7b49ef7d46fe725", "commit_rev":
"3e5152f0ec19bdbc37aa171be11690f86a6ab311"}

[commit-bot: I haz the power, 2017-01-09 07:14:58]

Description was changed from

==========
Store registered property InterpolationTypes in PropertyRegistration

This change moves the storage of InterpolationTypes for a registered
custom property to its PropertyRegistration object.
This is in preparation for supporting smooth interpolations on
registered custom properties.

There are no behavioural changes made by this patch.

BUG=671904
==========

to

==========
Store registered property InterpolationTypes in PropertyRegistration

This change moves the storage of InterpolationTypes for a registered
custom property to its PropertyRegistration object.
This is in preparation for supporting smooth interpolations on
registered custom properties.

There are no behavioural changes made by this patch.

BUG=671904

Review-Url: https://codereview.chromium.org/2618673003
Cr-Commit-Position: refs/heads/master@{#442204}
Committed:
https://chromium.googlesource.com/chromium/src/+/3e5152f0ec19bdbc37aa171be116...
==========

[commit-bot: I haz the power, 2017-01-09 07:14:59]

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

[alancutter (OOO until 2018), 2017-01-16 00:23:52]

Belated replies that I forgot to send out. ):

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/PropertyRegistration.cpp (right):

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/PropertyRegistration.cpp:7: #include
"core/animation/CSSColorInterpolationType.h"
On 2017/01/09 at 05:07:23, Timothy Loh wrote:
> not used yet

Removed

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/Document.h (right):

https://codereview.chromium.org/2618673003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/Document.h:1299: const PropertyRegistry*
propertyRegistry() const;
On 2017/01/09 at 05:07:23, Timothy Loh wrote:
> needs a rebase, this exists now :O

Rebased.