Closed Bug 1409596 Opened 7 years ago Closed 7 years ago

Don't pass input word as context data for matching XUL tree pseudos

Categories

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

enhancement

Tracking

()

RESOLVED INVALID
Tracking Status
firefox57 --- wontfix
firefox58 --- affected

People

(Reporter: xidorn, Assigned: xidorn)

References

Details

Currently in Gecko, we pass input word as context data for matching XUL tree pseudos. Implementing this mechanism seems to be troublesome in Stylo (see bug 1397644 comment 3).

Since we already depend on frame being created in this case, we can probably just get the array via the element.
So, this is probably not doable either, or at least it would be very hacky, because we can be calling into resolving tree pseudo style during early stage of frame initialization when the frame hasn't become the primary frame of element.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Maybe we can still try this... it seems that when trying to resolve style in early stage of frame initialization, mScratchArray is always cleared. We can assert that somewhere and use the element to query.

Actually it seems to me that the whole design of mScratchArray is quite inefficient...
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Alternatively we can put the pointer of the array to some global variable... (which is rather ugly...)
Priority: -- → P3
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.