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 global fields from DOM objects #9255
Conversation
As mentioned on IRC, please rename method to just |
Please squash the two last commits together and rename the method. Apart from that, r=me. -S-awaiting-review +S-needs-code-changes Reviewed 2 of 2 files at r1, 1 of 1 files at r2, 1 of 1 files at r3, 1 of 1 files at r4, 1 of 1 files at r5, 1 of 1 files at r6, 1 of 1 files at r7, 1 of 1 files at r8, 1 of 1 files at r9, 1 of 1 files at r10, 1 of 1 files at r11, 16 of 16 files at r12. components/script/dom/bindings/reflector.rs, line 80 [r11] (raw file): Comments from the review on Reviewable.io |
eb4907a
to
b5a5c10
Compare
b5a5c10
to
ce60758
Compare
-S-awaiting-review @bors-servo r=nox Thanks for working on this! Reviewed 1 of 2 files at r1, 17 of 17 files at r13. Comments from the review on Reviewable.io |
📌 Commit ce60758 has been approved by |
Remove global fields from DOM objects Fixes #9227 <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9255) <!-- Reviewable:end -->
☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt |
Fixes #9227