stylo: Add support for (chrome-only) document state selectors

RESOLVED DUPLICATE of bug 1390694

Status

()

Core
CSS Parsing and Computation
P4
normal
RESOLVED DUPLICATE of bug 1390694
a year ago
22 days ago

People

(Reporter: Bobby Holley (parental leave - send mail for anything urgent), Unassigned)

Tracking

(Blocks: 2 bugs)

unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 wontfix)

Details

Chrome documents support "document state selectors", which the servo style system doesn't know about. This isn't an issue for web content, but presumably blocks rendering chrome documents with stylo.
Blocks: 1294570
Priority: -- → P4
So this blocks :-moz-window-inactive, which is usable from content (although I don't think it should, but...).
Blocks: 1390694
(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #1)
> So this blocks :-moz-window-inactive, which is usable from content (although
> I don't think it should, but...).

Oh hm, that's pretty unfortunate. Is -moz-window-inactive something we definitely need to support in content? I guess maybe so?
(In reply to Bobby Holley (:bholley) (busy with Stylo) from comment #2)
> (In reply to Xidorn Quan [:xidorn] UTC+10 from comment #1)
> > So this blocks :-moz-window-inactive, which is usable from content (although
> > I don't think it should, but...).
> 
> Oh hm, that's pretty unfortunate. Is -moz-window-inactive something we
> definitely need to support in content? I guess maybe so?

Probably not too much. Also, since this requires changes to the invalidation map (at least if we want to do it efficiently, which I guess we do), I'd prefer to wait until I'm done with bug 1386045.
Depends on: 1386045
I don't think we should unship :-moz-window-inactive at this stage, but I don't think we should put lots of effort to have it work perfectly either, unless proven otherwise (e.g. some SVG files in chrome rely on this, or some websites use this as a way to detect user action, and is not usable if it doesn't work as expected).

That's also why I gave bug 1390694 P3 initially.
status-firefox57=wontfix unless someone thinks this bug should block 57
status-firefox57: --- → wontfix
Looks like there are only :-moz-window-inactive and :-moz-locale-dir two pseudo-classes. Both of them will be addressed in bug 1390694 so let's dupe this one to that bug.
Status: NEW → RESOLVED
Last Resolved: 22 days ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1390694
You need to log in before you can comment on or make changes to this bug.