Compute the linkMatchType inside the CSSSelectorParser.

Compute the linkMatchType inside the CSSSelectorParser.

Store the linkMatchType on the first CSSSelector in each CSSSelectorList
instead of computing it every time we build the RuleSets.

R=timloh@chromium.org
BUG=585651

[esprehn, 2015-06-02 03:22:32]

esprehn@chromium.org changed reviewers:
+ rune@opera.com

[esprehn, 2015-06-02 03:54:00]

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

[commit-bot: I haz the power, 2015-06-02 03:54:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1149913008/1

[Timothy Loh, 2015-06-02 04:20:59]

lgtm

https://codereview.chromium.org/1149913008/diff/1/Source/core/css/parser/CSSS...
File Source/core/css/parser/CSSSelectorParser.cpp (right):

https://codereview.chromium.org/1149913008/diff/1/Source/core/css/parser/CSSS...
Source/core/css/parser/CSSSelectorParser.cpp:94:
selector->updateLinkMatchType();
Probably better to be in consumeComplexSelector?

[esprehn, 2015-06-02 05:07:27]

On 2015/06/02 at 04:20:59, timloh wrote:
> lgtm
> 
>
https://codereview.chromium.org/1149913008/diff/1/Source/core/css/parser/CSSS...
> File Source/core/css/parser/CSSSelectorParser.cpp (right):
> 
>
https://codereview.chromium.org/1149913008/diff/1/Source/core/css/parser/CSSS...
> Source/core/css/parser/CSSSelectorParser.cpp:94:
selector->updateLinkMatchType();
> Probably better to be in consumeComplexSelector?

Yeah.

[esprehn, 2015-06-02 05:07:31]

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

[esprehn, 2015-06-02 05:07:31]

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/1149913008/#ps20001
(title: "Do it inside consumeComplexSelector.")

[commit-bot: I haz the power, 2015-06-02 05:08:11]

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

[commit-bot: I haz the power, 2015-06-02 06:50:54]

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

[commit-bot: I haz the power, 2015-06-02 06:50:55]

Dry run: Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/6...)

[esprehn, 2016-02-10 00:30:31]

Description was changed from

==========
Compute the linkMatchType inside the CSSSelectorParser.

Store the linkMatchType on the first CSSSelector in each CSSSelectorList
instead of computing it every time we build the RuleSets.

R=timloh@chromium.org
==========

to

==========
Compute the linkMatchType inside the CSSSelectorParser.

Store the linkMatchType on the first CSSSelector in each CSSSelectorList
instead of computing it every time we build the RuleSets.

R=timloh@chromium.org
BUG=585651
==========