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: Fix various issues with XBL rule matching #18652
Conversation
…n't have rules for it. This could prevent an XBL binding to use some pseudo-elements fairly randomly. MozReview-Commit-ID: b44DO8qcfD Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
Heads up! This PR modifies the following files:
|
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=TYLin
components/style/stylist.rs
Outdated
applicable_declarations, | ||
); | ||
let cut_off_inheritance = element.each_xbl_stylist(|stylist| { | ||
// nsXBLPrototypeResources::LoadResources() loads Chrome XBL style |
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.
While you're here, please s/nsXBLPrototypeResources::LoadResources()/ ServoStyleSet::CreateXBLServoStyleSet()/
, and delete "Chrome" in the comment because author XBL stylesheets also added under eAuthorSheetFeatures level.
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.
Done, thanks for the review! :)
We were using the wrong MatchingMode and flags setter just because we didn't bother threading them down. This patch fixes the issue seen with the video controls. MozReview-Commit-ID: Il1WOzRDxI1 Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
@bors-servo r=TYLin |
📌 Commit 19d34b9 has been approved by |
style: Fix various issues with XBL rule matching See the commit details. <!-- 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/18652) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
This commit caused https://bugzilla.mozilla.org/show_bug.cgi?id=1405526 |
See the commit details.
This change is