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
Remove the global argument to EventTarget::{fire_event, fire_simple_event}. #9571
Conversation
Ms2ger
commented
Feb 8, 2016
Yay, the code got a lot cleaner! Reviewed 9 of 9 files at r1. Comments from the review on Reviewable.io |
@bors-servo: r+ |
📌 Commit 5317af1 has been approved by |
Remove the global argument to EventTarget::{fire_event, fire_simple_event}. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9571) <!-- Reviewable:end -->
💔 Test failed - linux-rel |
The CSS failure is probably just an intermittent. The compiler warning is more of a concern, do you know what's causing it? |
Which compiler warning are you talking about? |
⚡ Previous build results for android, gonk, linux-dev, mac-dev-unit, mac-rel-css, mac-rel-wpt are reusable. Rebuilding only linux-rel... |
☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt |
The warnings at http://build.servo.org/builders/linux-rel/builds/1822, but if you reckon those are unrelated then yay. |