-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
style: Account for cousin sharing in the revalidation logic. #16670
Conversation
Heads up! This PR modifies the following files:
|
359f9e1
to
fde41bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me with that changed. Thanks for fixing this!
@@ -375,10 +376,7 @@ impl Stylist { | |||
#[inline] | |||
fn note_for_revalidation(&mut self, selector: &Selector<SelectorImpl>) { | |||
if needs_revalidation(selector) { | |||
// For revalidation, we can skip everything left of the first ancestor | |||
// combinator. | |||
let revalidation_sel = selector.inner.slice_to_first_ancestor_combinator(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably get rid of the slice_to_first_ancestor_combinator() API while we're at it. I don't think we use it for anything else.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I'd bet I had done that... Probably stashed it on the way, huh. Will do
@bors-servo delegate+ |
✌️ @emilio can now approve this pull request |
fde41bf
to
f879140
Compare
@bors-servo r=bholley |
📌 Commit f879140 has been approved by |
style: Account for cousin sharing in the revalidation logic. Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1361013 <!-- 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/16670) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-msvc-dev |
Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1361013
This change is