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
Partial fix for #12415: expose interfaces of some HTML Elements #12617
Conversation
Heads up! This PR modifies the following files:
|
Excellent, thank you! |
@bors-servo r+ |
📌 Commit 0eb907b has been approved by |
Partial fix for #12415: expose interfaces of some HTML Elements <!-- Please describe your changes on the following line: --> Fix for interfaces: HTMLHeadingElement.webidl HTMLHRElement.webidl HTMLHtmlElement.webidl HTMLHyperlinkElementUtils.webidl HTMLIFrameElement.webidl HTMLImageElement.webidl HTMLInputElement.webidl HTMLLabelElement.webidl HTMLLegendElement.webidl HTMLLIElement.webidl --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12415 <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/12617) <!-- Reviewable:end -->
💔 Test failed - linux-rel |
|
Should I remove the test? Do my changes really make it pass? |
It's a test that looks at properties of objects in workers, so it seems reasonable, yes. |
0eb907b
to
6d532cf
Compare
unexpected-self-properties.html removed |
Er, you removed the test itself; only the expectation ( |
OK, I'll bring it back :-) |
6d532cf
to
9fa6d50
Compare
Thank you kindly @bors-servo r+ |
📌 Commit 9fa6d50 has been approved by |
Partial fix for #12415: expose interfaces of some HTML Elements <!-- Please describe your changes on the following line: --> Fix for interfaces: HTMLHeadingElement.webidl HTMLHRElement.webidl HTMLHtmlElement.webidl HTMLHyperlinkElementUtils.webidl HTMLIFrameElement.webidl HTMLImageElement.webidl HTMLInputElement.webidl HTMLLabelElement.webidl HTMLLegendElement.webidl HTMLLIElement.webidl --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12415 <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/12617) <!-- Reviewable:end -->
⌛ Testing commit 9fa6d50 with merge 90243de... |
💔 Test failed - linux-rel |
Partial fix for #12415: expose interfaces of some HTML Elements <!-- Please describe your changes on the following line: --> Fix for interfaces: HTMLHeadingElement.webidl HTMLHRElement.webidl HTMLHtmlElement.webidl HTMLHyperlinkElementUtils.webidl HTMLIFrameElement.webidl HTMLImageElement.webidl HTMLInputElement.webidl HTMLLabelElement.webidl HTMLLegendElement.webidl HTMLLIElement.webidl --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #12415 <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/12617) <!-- Reviewable:end -->
☀️ Test successful - arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev |
Partial fix for #12415: expose WebGL interfaces <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes partially fix #12617 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because they do not add any features <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/13493) <!-- Reviewable:end -->
Fix for interfaces:
HTMLHeadingElement.webidl
HTMLHRElement.webidl
HTMLHtmlElement.webidl
HTMLHyperlinkElementUtils.webidl
HTMLIFrameElement.webidl
HTMLImageElement.webidl
HTMLInputElement.webidl
HTMLLabelElement.webidl
HTMLLegendElement.webidl
HTMLLIElement.webidl
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is