Skip to content
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: Ensure that precomputed pseudo-element declarations are correctly sorted. #15459

Merged
merged 1 commit into from Feb 9, 2017

Conversation

@emilio
Copy link
Member

emilio commented Feb 8, 2017


// We need to insert important rules _after_ normal rules for this to be
// correct, and also to not trigger rule tree assertions.

This comment has been minimized.

@emilio

emilio Feb 8, 2017

Author Member

I guess only to be correct is enough, but... :)

@SimonSapin
Copy link
Member

SimonSapin commented Feb 9, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

📌 Commit e9167fb has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

Testing commit e9167fb with merge 7246197...

bors-servo added a commit that referenced this pull request Feb 9, 2017
style: Ensure that precomputed pseudo-element declarations are correctly sorted.

Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1337657

r? @SimonSapin

<!-- 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/15459)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

💔 Test failed - linux-dev

@SimonSapin
Copy link
Member

SimonSapin commented Feb 9, 2017

error: method `get_universal_rules` is private
   --> /home/servo/buildbot/slave/linux-dev/build/tests/unit/style/stylist.rs:117:9
    |
117 |     map.get_universal_rules(&mut decls,
    |         ^^^^^^^^^^^^^^^^^^^

error: aborting due to previous error

error: Could not compile `style_tests`.
…tly sorted.
@emilio emilio force-pushed the emilio:ua-rules branch from e9167fb to db26ef9 Feb 9, 2017
@emilio
Copy link
Member Author

emilio commented Feb 9, 2017

Whoops.

@bors-servo r=SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

📌 Commit db26ef9 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

Testing commit db26ef9 with merge 6d1836e...

bors-servo added a commit that referenced this pull request Feb 9, 2017
style: Ensure that precomputed pseudo-element declarations are correctly sorted.

Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1337657

r? @SimonSapin

<!-- 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/15459)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

💔 Test failed - linux-rel-wpt

@emilio
Copy link
Member Author

emilio commented Feb 9, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

Previous build results for android, arm32, arm64, linux-dev, linux-rel-css, mac-dev-unit, mac-rel-wpt1, mac-rel-wpt2, windows-gnu-dev, windows-msvc-dev are reusable. Rebuilding only linux-rel-wpt, mac-rel-css...

@bors-servo
Copy link
Contributor

bors-servo commented Feb 9, 2017

☀️ 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-gnu-dev, windows-msvc-dev
Approved by: SimonSapin
Pushing 6d1836e to master...

@bors-servo bors-servo merged commit db26ef9 into servo:master Feb 9, 2017
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@emilio emilio deleted the emilio:ua-rules branch Feb 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.