1632647 - :is() and :where() shouldn't parse combinators in :host selectors.

Status: RESOLVED FIXED

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

Description

[Emilio Cobos Álvarez (:emilio)]

:is() and :where() I believe right now allow you to write:

:host(:is(div .foo))

Which breaks encapsulation of the shadow tree and is wrong. We should probably not allow combinators in there (or never match, I guess, though not allowing it seems more consistent with what :host already does).

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1632647 - Fix parsing of :is() and :where() to account for constraints from parent selectors. r=#style

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/df17138614f6
Fix parsing of :is() and :where() to account for constraints from parent selectors. r=heycam

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/23711 for changes under testing/web-platform/tests

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Can't merge web-platform-tests PR due to failing upstream checks:
Github PR https://github.com/web-platform-tests/wpt/pull/23711
* Community-TC (pull_request) (https://community-tc.services.mozilla.com/tasks/groups/LQYc9Dq3ReudyAlo0xb2Zw)

Comment 6

[Emilio Cobos Álvarez (:emilio)]

Seems like an error downloading chromedriver :)

Comment 7

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/df17138614f6

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot