Clean up CSSParserContext arguments in property parsing code

Clean up CSSParserContext arguments in property parsing code

This patch cleans up CSSParserContext arguments in the property parser,
tightening them to the CSSParserMode if possible, and replacing values
by const refs to avoid unneeded copies.

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

[Timothy Loh, 2015-12-10 04:01:49]

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

[Eric Willigers, 2015-12-10 04:11:58]

lgtm

[rwlbuis, 2015-12-10 04:17:55]

Lgtm.

Btw, are you sure this is compatible with the CSS Property API patch? AFAIK that
one want CSSParserContext in order to be more flexible.

[Timothy Loh, 2015-12-10 04:17:57]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2015-12-10 04:18:27]

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

[Timothy Loh, 2015-12-10 04:19:08]

On 2015/12/10 04:17:55, rwlbuis wrote:
> Lgtm.
> 
> Btw, are you sure this is compatible with the CSS Property API patch? AFAIK
that
> one want CSSParserContext in order to be more flexible.

Should be, you can always go from a parser context to a parser mode so I don't
think there's a problem.

[commit-bot: I haz the power, 2015-12-10 05:50:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-10 05:51:31]

Description was changed from

==========
Clean up CSSParserContext arguments in property parsing code

This patch cleans up CSSParserContext arguments in the property parser,
tightening them to the CSSParserMode if possible, and replacing values
by const refs to avoid unneeded copies.

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

to

==========
Clean up CSSParserContext arguments in property parsing code

This patch cleans up CSSParserContext arguments in the property parser,
tightening them to the CSSParserMode if possible, and replacing values
by const refs to avoid unneeded copies.

BUG=499780

Committed: https://crrev.com/fd8931ee10c1f9ed2bd48deca4b5eec27dd6adbd
Cr-Commit-Position: refs/heads/master@{#364289}
==========

[commit-bot: I haz the power, 2015-12-10 05:51:32]

Patchset 1 (id:??) landed as
https://crrev.com/fd8931ee10c1f9ed2bd48deca4b5eec27dd6adbd
Cr-Commit-Position: refs/heads/master@{#364289}