Page MenuHomePhabricator

Bug 1600773 - Invalidate shadow part pseudo-class styles correctly. r=heycam
ClosedPublic

Authored by emilio on Dec 3 2019, 1:36 AM.

Details

Summary

I was going to send a test for :focus via wpt, but then realized it was
probably not spec-compliant with the new rules people want to follow for
:focus, so I filed https://github.com/w3c/csswg-drafts/issues/4555 instead.

Testing :hover / :active via wpt looked quite a bit of a hassle.

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

emilio created this revision.Dec 3 2019, 1:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 3 2019, 1:36 AM
phab-bot requested review of this revision.Dec 3 2019, 1:36 AM
phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: Restricted Project.
bgrins added a subscriber: bgrins.Dec 3 2019, 3:51 AM
heycam accepted this revision.Dec 9 2019, 4:03 AM
heycam added inline comments.
servo/components/style/dom.rs
854

Nit: drop one blank line?

This revision is now accepted and ready to land.Dec 9 2019, 4:03 AM
emilio updated this revision to Diff 204727.Dec 9 2019, 1:40 PM

Remove extra newline