Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor
and isValidNumericValue make more sense in StyleColor and
CSSParserToken respectively.
Also remove a superfluous std::isfinite check.

Committed: https://crrev.com/7c190d0749ed473cdd9dea6c29181336306c9308
Cr-Commit-Position: refs/heads/master@{#399215}

[rwlbuis, 2016-06-09 23:37:26]

Description was changed from

==========
SystemColor WIP

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

to

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, systemColor
==========

[rwlbuis, 2016-06-09 23:39:14]

Description was changed from

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, systemColor
==========

to

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor and
isValidNumericValue make more sense in StyleColor and CSSParserToken
respectively.
Also remove a superfluous std::isfinite check.
==========

[rwlbuis, 2016-06-09 23:39:48]

Description was changed from

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor and
isValidNumericValue make more sense in StyleColor and CSSParserToken
respectively.
Also remove a superfluous std::isfinite check.
==========

to

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor
and isValidNumericValue make more sense in StyleColor and
CSSParserToken respectively.
Also remove a superfluous std::isfinite check.
==========

[rwlbuis, 2016-06-09 23:40:49]

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

[rwlbuis, 2016-06-09 23:40:50]

PTAL.

[Timothy Loh, 2016-06-10 00:56:56]

On 2016/06/09 23:40:50, rwlbuis wrote:
> PTAL.

lgtm, although for the numeric range stuff maybe it'd better to just rebase and
fix up https://codereview.chromium.org/1914693002/

[rwlbuis, 2016-06-10 13:06:43]

On 2016/06/10 00:56:56, Timothy Loh wrote:
> On 2016/06/09 23:40:50, rwlbuis wrote:
> > PTAL.
> 
> lgtm, although for the numeric range stuff maybe it'd better to just rebase
and
> fix up https://codereview.chromium.org/1914693002/

I'll be happy to work on that one, for now there is no harm with this CL though
:)

[rwlbuis, 2016-06-10 13:06:49]

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

[commit-bot: I haz the power, 2016-06-10 13:07:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2056593005/20001

[commit-bot: I haz the power, 2016-06-10 17:14:27]

Description was changed from

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor
and isValidNumericValue make more sense in StyleColor and
CSSParserToken respectively.
Also remove a superfluous std::isfinite check.
==========

to

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor
and isValidNumericValue make more sense in StyleColor and
CSSParserToken respectively.
Also remove a superfluous std::isfinite check.
==========

[commit-bot: I haz the power, 2016-06-10 17:14:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-10 17:14:38]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-10 17:15:59]

Description was changed from

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor
and isValidNumericValue make more sense in StyleColor and
CSSParserToken respectively.
Also remove a superfluous std::isfinite check.
==========

to

==========
Move more helper methods out of CSSPropertyParser

Move more helper methods out of CSSPropertyParser, isSystemColor
and isValidNumericValue make more sense in StyleColor and
CSSParserToken respectively.
Also remove a superfluous std::isfinite check.

Committed: https://crrev.com/7c190d0749ed473cdd9dea6c29181336306c9308
Cr-Commit-Position: refs/heads/master@{#399215}
==========

[commit-bot: I haz the power, 2016-06-10 17:15:59]

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