diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2017-12-14 00:22:45 -0600 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-12-14 00:22:45 -0600 |
commit | f5129ef7b5efa59a176cac98dc68d910cb3279c9 (patch) | |
tree | bb1a2d1f98a5dd7de43ca4e3ad714b77c3a1d082 /components/layout_thread/lib.rs | |
parent | c258bfb430fabc2ebc55db679a52f09c7df04b58 (diff) | |
parent | 7886e033aa7e813a9ed3becc456ee9ed059364cc (diff) | |
download | servo-f5129ef7b5efa59a176cac98dc68d910cb3279c9.tar.gz servo-f5129ef7b5efa59a176cac98dc68d910cb3279c9.zip |
Auto merge of #19541 - emilio:parse-slotted, r=heycam
selectors: Add parsing support for ::slotted().
Without turning it on yet, of course.
The reason why I didn't use the general PseudoElement mechanism is because this
pseudo is a bit of its own thing, and I found easier to make ::selectors know
about it (because you need to jump to the assigned slot) than the other way
around.
Also, we need to support ::slotted(..)::before and such, and supporting multiple
pseudo-elements like that breaks some other invariants around the SelectorMap,
and fixing those would require special-casing slotted a lot more in other parts
of the code.
Let me know if you think otherwise.
I also don't like much the boolean tuple return value, but I plan to do some
cleanup in the area in a bit, so it should go away soon, I'd hope.
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/19541)
<!-- Reviewable:end -->
Diffstat (limited to 'components/layout_thread/lib.rs')
0 files changed, 0 insertions, 0 deletions