1893923 - Add ChromeOnly CSSStyleRule::querySelectorAll

Status: RESOLVED FIXED

(DevTools :: Inspector: Rules, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

This will take a root node, and will return the elements inside the root node that match the rule's selectors
This way, we won't have to serialize the rule (desugared) selectorText which can be expensive

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1893923 - [devtools] Add ChromeOnly CSSStyleRule::querySelectorAll. r=#layout!,#devtools-reviewers!.

For the selector highlighter, we were retrieving the desugared selector of each
displayed rule, and using the selector text in querySelectorAll to retrieve the
elements matching the rule.
This can be very expensive, especially for deeply nested rule, for a feature that
might not even be used.
This patch is adding a method which takes a root node, and will return the
elements inside the root node that match the rule's selectors.
We're only exposing the method that existed in glue.rs to get the SelectorList
of a given Rule, and call Servo_SelectorList_QueryAll with it to get our NodeList.

A test file is added to ensure this works as expected.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dbd850873912
[devtools] Add ChromeOnly CSSStyleRule::querySelectorAll. r=layout-reviewers,emilio.

Comment 3

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/dbd850873912