Update getComputedStyle with unknown pseudos to the spec.
Categories
(Core :: CSS Parsing and Computation, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox92 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 2 open bugs)
Details
Attachments
(3 files)
Per https://github.com/w3c/csswg-drafts/issues/2149 we should throw on invalid pseudo-elements. Let's try this behavior on Nightly.
Assignee | ||
Comment 1•3 years ago
|
||
This will make implementing the new behavior behind a pref
really straight-forward, and is generally nicer.
Assignee | ||
Comment 2•3 years ago
|
||
I need to update tests, try incoming.
Depends on D121705
Assignee | ||
Comment 3•3 years ago
|
||
With the patch above, some devtools tests failed because we were trying
to call getComputedStyle(node, ":marker") (read: one colon rather than
two).
Using two colons for pseudo-elements is the right thing to do and fixes
it / removes some weird special-cases.
Assignee | ||
Updated•3 years ago
|
Comment 6•3 years ago
|
||
Backed out 2 changesets (bug 1723921) for Mochitest failures in layout/inspector/tests/test_getCSSPseudoElementNames.html. CLOSED TREE
Log:
https://treeherder.mozilla.org/logviewer?job_id=347536827&repo=autoland&lineNumber=2327
https://treeherder.mozilla.org/logviewer?job_id=347536605&repo=autoland&lineNumber=1986
Push with failures:
https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&revision=e55c2177bbd2b98bd8005a7e5423bdae165ffaef
Backout:
https://hg.mozilla.org/integration/autoland/rev/0c7d036ab79c4bb37ad3c38ae5232c47e209eb84
Assignee | ||
Comment 7•3 years ago
|
||
Yeah, this depends on the other patch to work.
Comment 11•3 years ago
|
||
bugherder |
Assignee | ||
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Description
•