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

Use the select macro on the paint thread. #9493

Merged
merged 1 commit into from Feb 3, 2016

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Feb 1, 2016

Review on Reviewable

@highfive
Copy link

highfive commented Feb 1, 2016

warning Warning warning

  • These commits modify gfx code, but no tests are modified. Please consider adding a test!
@jdm
Copy link
Member

jdm commented Feb 1, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Feb 1, 2016

📌 Commit 2fd53c2 has been approved by jdm

@jdm jdm self-assigned this Feb 1, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2016

Testing commit 2fd53c2 with merge 31fb66c...

bors-servo added a commit that referenced this pull request Feb 2, 2016
Use the select macro on the paint thread.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9493)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2016

💔 Test failed - mac-rel-wpt

@jdm
Copy link
Member

jdm commented Feb 2, 2016

@nox nox added S-awaiting-merge and removed S-tests-failed labels Feb 2, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Feb 2, 2016

Testing commit 2fd53c2 with merge 3c7b703...

bors-servo added a commit that referenced this pull request Feb 2, 2016
Use the select macro on the paint thread.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9493)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 3, 2016

💔 Test failed - linux-rel

@jdm
Copy link
Member

jdm commented Feb 3, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Feb 3, 2016

Previous build results for android, gonk, linux-dev, mac-dev-unit, mac-rel-wpt are reusable. Rebuilding only linux-rel, mac-rel-css...

@nox nox added S-awaiting-merge and removed S-tests-failed labels Feb 3, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Feb 3, 2016

💔 Test failed - linux-rel

@nox
Copy link
Member

nox commented Feb 3, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Feb 3, 2016

Testing commit 2fd53c2 with merge 2d4a5eb...

bors-servo added a commit that referenced this pull request Feb 3, 2016
Use the select macro on the paint thread.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9493)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 3, 2016

@bors-servo bors-servo merged commit 2fd53c2 into servo:master Feb 3, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Ms2ger Ms2ger deleted the Ms2ger:paint-thread-messages branch Feb 19, 2016
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

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