Refactored out need to pass property to IsValidAnimationPropertyList.

Refactored out need to pass property to IsValidAnimationPropertyList.

Part of project Ribbon

This refactor is one step in the process of refactoring out property
specific logic inside switch cases in
CSSPropertyParser::ParseSingleValue. This is required to implement the
parseSingleValue methods in the property APIs without passing property
ID to the API.

Given that the IsValidAnimationPropertyList function only ever returned
false if the property passed was CSSPropertyTransitionProperty, this
check was moved to the calling methods.

The next step in this process will be to refactor out the need to pass
property to the ConsumeAnimationPropertyList function. At this stage the
call to IsValidAnimationPropertyList will be moved to the
CSSPropertyTransitionProperty case only and the check for this property
will be removed.

The check for this property in
CSSPropertyParser::ConsumeAnimationShorthand will be refactored out at a
later stage in project Ribbon.

BUG=668012
Review-Url: https://codereview.chromium.org/2835893002
Cr-Commit-Position: refs/heads/master@{#467277}
Committed: https://chromium.googlesource.com/chromium/src/+/b58fb7956f56025d0cf384376995ad54f39b6c78

[Bugs Nash, 2017-04-24 03:20:47]

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

[commit-bot: I haz the power, 2017-04-24 03:20:54]

Dry run: CQ is trying da patch.

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

[Bugs Nash, 2017-04-24 03:22:58]

bugsnash@chromium.org changed reviewers:
+ nainar@chromium.org

[commit-bot: I haz the power, 2017-04-24 04:56:32]

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

[commit-bot: I haz the power, 2017-04-24 04:56:33]

Dry run: This issue passed the CQ dry run.

[nainar, 2017-04-24 05:15:56]

lgtm

[Bugs Nash, 2017-04-24 05:33:01]

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

[Bugs Nash, 2017-04-24 05:33:02]

+alancutter for owners

[alancutter (OOO until 2018), 2017-04-26 04:52:26]

lgtm

[Bugs Nash, 2017-04-26 07:12:01]

The CQ bit was checked by bugsnash@chromium.org

[commit-bot: I haz the power, 2017-04-26 07:12: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-04-26 09:07:46]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493190721382510, "parent_rev":
"40c92451bdce5b9b9fae80f7bdd87ffc9b74e16e", "commit_rev":
"b58fb7956f56025d0cf384376995ad54f39b6c78"}

[commit-bot: I haz the power, 2017-04-26 09:07:53]

Description was changed from

==========
Refactored out need to pass property to IsValidAnimationPropertyList.

Part of project Ribbon

This refactor is one step in the process of refactoring out property
specific logic inside switch cases in
CSSPropertyParser::ParseSingleValue. This is required to implement the
parseSingleValue methods in the property APIs without passing property
ID to the API.

Given that the IsValidAnimationPropertyList function only ever returned
false if the property passed was CSSPropertyTransitionProperty, this
check was moved to the calling methods.

The next step in this process will be to refactor out the need to pass
property to the ConsumeAnimationPropertyList function. At this stage the
call to IsValidAnimationPropertyList will be moved to the
CSSPropertyTransitionProperty case only and the check for this property
will be removed.

The check for this property in
CSSPropertyParser::ConsumeAnimationShorthand will be refactored out at a
later stage in project Ribbon.

BUG=668012
==========

to

==========
Refactored out need to pass property to IsValidAnimationPropertyList.

Part of project Ribbon

This refactor is one step in the process of refactoring out property
specific logic inside switch cases in
CSSPropertyParser::ParseSingleValue. This is required to implement the
parseSingleValue methods in the property APIs without passing property
ID to the API.

Given that the IsValidAnimationPropertyList function only ever returned
false if the property passed was CSSPropertyTransitionProperty, this
check was moved to the calling methods.

The next step in this process will be to refactor out the need to pass
property to the ConsumeAnimationPropertyList function. At this stage the
call to IsValidAnimationPropertyList will be moved to the
CSSPropertyTransitionProperty case only and the check for this property
will be removed.

The check for this property in
CSSPropertyParser::ConsumeAnimationShorthand will be refactored out at a
later stage in project Ribbon.

BUG=668012

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

[commit-bot: I haz the power, 2017-04-26 09:07:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b58fb7956f56025d0cf384376995...