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
Don't try to share style on quirks mode whenever two elements have different id #19429
Conversation
Heads up! This PR modifies the following files:
|
@bors-servo r=emilio |
📌 Commit af8026a has been approved by |
Don't try to share style on quirks mode whenever two elements have different id This fixes [bug 1420946](https://bugzilla.mozilla.org/show_bug.cgi?id=1420946). <!-- 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/19429) <!-- Reviewable:end -->
💔 Test failed - linux-rel-wpt |
@bors-servo retry |
⚡ Previous build results for android, arm32, arm64, linux-dev, linux-rel-css, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev are reusable. Rebuilding only linux-rel-wpt... |
💔 Test failed - linux-rel-wpt |
Interesting... looks like this fixes tons of wpt in servo... How can this happen? |
That's surprising, specially because most of those tests don't even have elements with |
@bors-servo retry |
I'll investigate a bit locally. |
The tests definitely do not pass locally, I cannot make any sense of that failure. |
Don't try to share style on quirks mode whenever two elements have different id This fixes [bug 1420946](https://bugzilla.mozilla.org/show_bug.cgi?id=1420946). <!-- 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/19429) <!-- 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 fixes bug 1420946.
This change is