Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-points-x and
scroll-snap-points-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780
Committed: https://crrev.com/2bc4a11520e612df3855fbfdf55c2035441f2b82
Cr-Commit-Position: refs/heads/master@{#368898}

[rwlbuis, 2016-01-11 22:45:37]

Description was changed from

==========
snap WIP

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

to

==========
Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-point-x and
scroll-snap-point-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780
==========

[rwlbuis, 2016-01-11 22:46:13]

Description was changed from

==========
Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-point-x and
scroll-snap-point-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780
==========

to

==========
Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-points-x and
scroll-snap-points-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780
==========

[rwlbuis, 2016-01-11 22:48:17]

rob.buis@samsung.com changed reviewers:
+ timloh@chromium.org

[rwlbuis, 2016-01-11 22:48:17]

PTAL. This removes some parsePosition related helpers from
LegacyCSSPropertyParser.cpp...

[Timothy Loh, 2016-01-12 05:19:01]

lgtm

https://codereview.chromium.org/1576163002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp (right):

https://codereview.chromium.org/1576163002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp:2917: if
(parsedValue && (parsedValue->isCalculated() || parsedValue->getDoubleValue() >
0)) {
missing args.atEnd() check

[rwlbuis, 2016-01-12 15:23:19]

The CQ bit was checked by rob.buis@samsung.com

[rwlbuis, 2016-01-12 15:23:20]

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/1576163002/#ps40001
(title: "Patch for landing")

[commit-bot: I haz the power, 2016-01-12 15:23:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1576163002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1576163002/40001

[commit-bot: I haz the power, 2016-01-12 16:48:02]

Description was changed from

==========
Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-points-x and
scroll-snap-points-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780
==========

to

==========
Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-points-x and
scroll-snap-points-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780
==========

[commit-bot: I haz the power, 2016-01-12 16:48:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-12 16:49:53]

Description was changed from

==========
Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-points-x and
scroll-snap-points-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780
==========

to

==========
Move scroll-snap related properties into CSSPropertyParser

Move scroll-snap-coordinate, scroll-snap-points-x and
scroll-snap-points-y from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780

Committed: https://crrev.com/2bc4a11520e612df3855fbfdf55c2035441f2b82
Cr-Commit-Position: refs/heads/master@{#368898}
==========

[commit-bot: I haz the power, 2016-01-12 16:49:54]

Patchset 3 (id:??) landed as
https://crrev.com/2bc4a11520e612df3855fbfdf55c2035441f2b82
Cr-Commit-Position: refs/heads/master@{#368898}