1365475 - stylo: Complex selectors with multiple class selectors end up doing multiple FFI calls to get the class attr

Status: NEW

(Core :: CSS Parsing and Computation, enhancement, P4)

Description

[Boris Zbarsky [:bzbarsky]]

The way things are set up right now, each class selector causes a has_class call, which will go looking for the "class" attribute.  It sure would be nice if we either cached the result somewhere or coalesced multiple class simple selectors or something...

Gecko doesn't have this problem, since it automatically coalesces class simple selectors into a single call to GetClasses followed by work with the resulting list-or-atom.

Comment 1

[Emilio Cobos Álvarez (:emilio)]

One idea to fix this is using an `ElementMatchingContext` or something like that, and pass it down to the loop where we go through the selector sequence. Shouldn't be too hard I think.

Comment 2

[Bobby Holley (:bholley)]

Yeah seems like we should just use an LRU cache for id and class, and stick that on the existing MatchingContext;

Comment 3

[Chris Peterson [:cpeterson]]

status-firefox57=wontfix unless someone thinks this bug should block 57