Refactored out property specific logic in animation list parsing.

Refactored out property specific logic in animation list parsing.

Replaced animation list parsing method ConsumeAnimationPropertyList
with a new more general list parsing method ConsumeCommaSeparatedList.
This removes the need to pass property information to the list parsing
method, instead passing a callback function to be called on each of the
items in the list.

This is pre work to implementing the parseSingleValue method in the
property APIs for the animation properties that expect a list, as the
APIs do not take property as an argument.

The new ConsumeCommaSeparatedList method can be used in other parts of
the code base, which may be done in future patches.

This patch
- Added ConsumeCommaSeparatedList templated function to
  CSSPropertyParserHelpers
- Made each animation list property in parseSingleValue use the new
  ConsumeCommaSeparatedList method instead of
  ConsumeAnimationPropertyList
- Deleted ConsumeAnimationPropertyList method

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

[Bugs Nash, 2017-05-02 03:22:05]

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

[commit-bot: I haz the power, 2017-05-02 03:22:34]

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-05-02 03:23:53]

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

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

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

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

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Bugs Nash, 2017-05-04 06:14:38]

Fixed crashing layout tests

[Bugs Nash, 2017-05-04 06:15:37]

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

[commit-bot: I haz the power, 2017-05-04 06:15:51]

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-05-04 07:52:12]

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

[commit-bot: I haz the power, 2017-05-04 07:52:13]

Dry run: This issue passed the CQ dry run.

[Bugs Nash, 2017-05-05 00:09:50]

bugsnash@chromium.org changed reviewers:
+ ericwilligers@chromium.org
- rjwright@chromium.org

[Eric Willigers, 2017-05-05 01:23:34]

lgtm

[Bugs Nash, 2017-05-05 01:39:36]

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

[Bugs Nash, 2017-05-05 01:39:37]

+suzyh for owners

[suzyh_UTC10 (ex-contributor), 2017-05-05 01:45:02]

lgtm

[Bugs Nash, 2017-05-05 01:46:02]

The CQ bit was checked by bugsnash@chromium.org

[commit-bot: I haz the power, 2017-05-05 01:46: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-05-05 01:53:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493948762725510,
"parent_rev": "a2a5081b066eb679783fc84c7fb4b11480ffb696", "commit_rev":
"8091ef8119fff447dc756b8dfc6e99b944a479f8"}

[commit-bot: I haz the power, 2017-05-05 01:54:07]

Description was changed from

==========
Refactored out property specific logic in animation list parsing.

Replaced animation list parsing method ConsumeAnimationPropertyList
with a new more general list parsing method ConsumeCommaSeparatedList.
This removes the need to pass property information to the list parsing
method, instead passing a callback function to be called on each of the
items in the list.

This is pre work to implementing the parseSingleValue method in the
property APIs for the animation properties that expect a list, as the
APIs do not take property as an argument.

The new ConsumeCommaSeparatedList method can be used in other parts of
the code base, which may be done in future patches.

This patch
- Added ConsumeCommaSeparatedList templated function to
  CSSPropertyParserHelpers
- Made each animation list property in parseSingleValue use the new
  ConsumeCommaSeparatedList method instead of
  ConsumeAnimationPropertyList
- Deleted ConsumeAnimationPropertyList method

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

to

==========
Refactored out property specific logic in animation list parsing.

Replaced animation list parsing method ConsumeAnimationPropertyList
with a new more general list parsing method ConsumeCommaSeparatedList.
This removes the need to pass property information to the list parsing
method, instead passing a callback function to be called on each of the
items in the list.

This is pre work to implementing the parseSingleValue method in the
property APIs for the animation properties that expect a list, as the
APIs do not take property as an argument.

The new ConsumeCommaSeparatedList method can be used in other parts of
the code base, which may be done in future patches.

This patch
- Added ConsumeCommaSeparatedList templated function to
  CSSPropertyParserHelpers
- Made each animation list property in parseSingleValue use the new
  ConsumeCommaSeparatedList method instead of
  ConsumeAnimationPropertyList
- Deleted ConsumeAnimationPropertyList method

BUG=668012

Review-Url: https://codereview.chromium.org/2849363002
Cr-Commit-Position: refs/heads/master@{#469569}
Committed:
https://chromium.googlesource.com/chromium/src/+/8091ef8119fff447dc756b8dfc6e...
==========

[commit-bot: I haz the power, 2017-05-05 01:54:08]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/8091ef8119fff447dc756b8dfc6e...