Closed
Bug 1374357
Opened 8 years ago
Closed 8 years ago
stylo: ensure we match invalidation selectors in the "fast" order, whichever that is
Categories
(Core :: CSS Parsing and Computation, enhancement)
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: bzbarsky, Unassigned)
References
(Blocks 1 open bug)
Details
See bug 1373800 comment 13. We should ensure that invalidation selectors are matches in the "fast" order, however that's implemented. Ideally we can do this in such a way that future changes to underlying storage order don't involve changes to invalidation selector matching code....
![]() |
Reporter | |
Comment 1•8 years ago
|
||
Bobby is this still a perf issue after the fix for bug 1373800? That is, is there more perf work to be done here, or is this just a "make the code more robust" bug at this point?
Flags: needinfo?(bobbyholley)
Comment 2•8 years ago
|
||
I don't know if there's much we can do here. Unless we want to use separate storage for invalidation selectors vs the original selector (I believe right now we just use indices), we can't have the fast order in both left-to-right and right-to-left matching modes.
So I think this is probably WONTFIX, unless I'm misunderstanding one of the pieces. NI emilio to confirm.
Flags: needinfo?(bobbyholley) → needinfo?(emilio+bugs)
Comment 3•8 years ago
|
||
Yeah, it's unclear if walking the whole partial selector and matching the other way would be faster. Intuitively it would not, but you never know.
I'd prefer not doing it if we don't know it's a bottleneck of any sort, given it would definitely make the invalidation stuff more complex (and I'd like to keep it as obvious as possible).
Flags: needinfo?(emilio+bugs)
Comment 4•8 years ago
|
||
Ok, resolving until we have a clearer need.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•