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

implement display_handler::on_status_message for cef #6890

Merged
merged 1 commit into from Aug 2, 2015

Conversation

@zmike
Copy link
Contributor

zmike commented Aug 1, 2015

@metajack
Copy link
Contributor

metajack commented Aug 1, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Aug 1, 2015

📌 Commit a03a0cb has been approved by metajack

@bors-servo
Copy link
Contributor

bors-servo commented Aug 1, 2015

Testing commit a03a0cb with merge 4c8ba07...

bors-servo pushed a commit that referenced this pull request Aug 1, 2015
implement display_handler::on_status_message for cef

r+? @jdm @larsbergstrom @metajack @pcwalton @anyone with r+

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

SimonSapin commented Aug 1, 2015

Homu or buildbot seems stuck here, trying to close/reopen to see if that helps.

@SimonSapin SimonSapin closed this Aug 1, 2015
@SimonSapin SimonSapin reopened this Aug 1, 2015
@bors-servo
Copy link
Contributor

bors-servo commented Aug 1, 2015

Testing commit a03a0cb with merge 69dd08e...

bors-servo pushed a commit that referenced this pull request Aug 1, 2015
implement display_handler::on_status_message for cef

r+? @jdm @larsbergstrom @metajack @pcwalton @anyone with r+

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

metajack commented Aug 1, 2015

@bors-servo retry

@metajack metajack closed this Aug 2, 2015
@metajack metajack reopened this Aug 2, 2015
@metajack
Copy link
Contributor

metajack commented Aug 2, 2015

@bors-servo retry force

@bors-servo
Copy link
Contributor

bors-servo commented Aug 2, 2015

Testing commit a03a0cb with merge 2e90540...

bors-servo pushed a commit that referenced this pull request Aug 2, 2015
implement display_handler::on_status_message for cef

r+? @jdm @larsbergstrom @metajack @pcwalton @anyone with r+

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6890)
<!-- Reviewable:end -->
@metajack metajack closed this Aug 2, 2015
@metajack metajack reopened this Aug 2, 2015
@metajack
Copy link
Contributor

metajack commented Aug 2, 2015

@metajack
Copy link
Contributor

metajack commented Aug 2, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Aug 2, 2015

Testing commit a03a0cb with merge 01281a4...

bors-servo pushed a commit that referenced this pull request Aug 2, 2015
implement display_handler::on_status_message for cef

r+? @jdm @larsbergstrom @metajack @pcwalton @anyone with r+

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

metajack commented Aug 2, 2015

@bors-servo retry force

@bors-servo
Copy link
Contributor

bors-servo commented Aug 2, 2015

Testing commit a03a0cb with merge 42a9756...

bors-servo pushed a commit that referenced this pull request Aug 2, 2015
implement display_handler::on_status_message for cef

r+? @jdm @larsbergstrom @metajack @pcwalton @anyone with r+

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

metajack commented Aug 2, 2015

@zmike You need to associate your commit email with GitHub or it will confuse our systems. I worked around this with servo/saltfs#92 for now.

@bors-servo
Copy link
Contributor

bors-servo commented Aug 2, 2015

☀️ Test successful - android, gonk, linux1, linux2, linux3, mac1, mac2, mac3

@bors-servo bors-servo merged commit a03a0cb into servo:master Aug 2, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
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.