Move legacy webkit properties into CSSPropertyParser

Move legacy webkit properties into CSSPropertyParser

Move legacy webkit related properties from
LegacyCSSPropertyParser into CSSPropertyParser.

BUG=499780
Committed: https://crrev.com/31cac25752d71e57922d1f17a0e819faea3585a4
Cr-Commit-Position: refs/heads/master@{#354609}

[rwlbuis, 2015-10-15 22:13:05]

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

[rwlbuis, 2015-10-15 22:13:05]

PTAL

[Timothy Loh, 2015-10-15 23:44:36]

lgtm, but it's probably a good idea to list the properties you're moving in the
description.

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

https://codereview.chromium.org/1405793002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp:840: if
(range.peek().type() != PercentageToken && range.peek().type() != NumberToken)
I wouldn't bother disallowing calcs, but I'll leave it up to you.

https://codereview.chromium.org/1405793002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp:917: // Apple
specific properties.  These will never be standardized and are purely to
Not sure we need this comment. -webkit-line-clamp has pretty high (~10%) usage
and IIRC we might be looking to standardize it.

[rwlbuis, 2015-10-16 20:08:57]

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

[rwlbuis, 2015-10-16 20:08:57]

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

[commit-bot: I haz the power, 2015-10-16 20:09:06]

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

[commit-bot: I haz the power, 2015-10-16 21:41:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-10-16 21:41:57]

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

[rwlbuis, 2015-10-19 13:47:42]

Patchset #4 (id:60001) has been deleted