Refactored out need to use CSSPropertyID to parse counter properties.

Refactored out need to use CSSPropertyID to parse counter properties.

Refactored out the need to use CSSPropertyID in the parsing logic of the
CSSPropertyCounterIncrement and CSSPropertyCounterReset properties. This
is pre work to allow the parseSingleValue method of these properties to
be implemented in their property APIs.

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

[Bugs Nash, 2017-03-27 04:15:24]

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

[commit-bot: I haz the power, 2017-03-27 04:15:32]

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-03-27 04:20:21]

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

[suzyh_UTC10 (ex-contributor), 2017-03-27 04:25:43]

lgtm

[suzyh_UTC10 (ex-contributor), 2017-03-27 04:29:57]

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

https://codereview.chromium.org/2777793002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp:1867: return
consumeCounter(m_range, 1);
These magic numbers should be replaced with meaningfully-named constants. Happy
for that to happen in the next step's patch.

[commit-bot: I haz the power, 2017-03-27 04:45:12]

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

[commit-bot: I haz the power, 2017-03-27 04:45:13]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Bugs Nash, 2017-03-27 04:53:05]

The CQ bit was checked by bugsnash@chromium.org

[commit-bot: I haz the power, 2017-03-27 04:53:15]

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-03-27 06:42:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490590385931850, "parent_rev":
"dac4459e32961172505fc30c395886c1236fe469", "commit_rev":
"24acb2c9fcaeef8e5b30b82e5b2fdaa9396353c7"}

[commit-bot: I haz the power, 2017-03-27 06:44:17]

Description was changed from

==========
Refactored out need to use CSSPropertyID to parse counter properties.

Refactored out the need to use CSSPropertyID in the parsing logic of the
CSSPropertyCounterIncrement and CSSPropertyCounterReset properties. This
is pre work to allow the parseSingleValue method of these properties to
be implemented in their property APIs.

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

to

==========
Refactored out need to use CSSPropertyID to parse counter properties.

Refactored out the need to use CSSPropertyID in the parsing logic of the
CSSPropertyCounterIncrement and CSSPropertyCounterReset properties. This
is pre work to allow the parseSingleValue method of these properties to
be implemented in their property APIs.

BUG=668012

Review-Url: https://codereview.chromium.org/2777793002
Cr-Commit-Position: refs/heads/master@{#459709}
Committed:
https://chromium.googlesource.com/chromium/src/+/24acb2c9fcaeef8e5b30b82e5b2f...
==========

[commit-bot: I haz the power, 2017-03-27 06:44:18]

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