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
Browser API: implement iframe.reload() #8618
Conversation
Is there a spec for how this is supposed to behave? Is it supposed to reload the current contents of the iframe, or reload the url specified by the src attribute? |
It's supposed to reload the current contents of the iframe… and now I realize that this code reloads the src attribute :) |
@bors-servo: r+ |
📌 Commit 2a86f9d has been approved by |
Browser API: implement iframe.reload() fixes #8575 The implementation is naive, and doesn't support the `hardreload` parameter. And for the test, I'm not sure how else I can test the reload. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8618) <!-- Reviewable:end -->
⛄ The build was interrupted to prioritize another pull request. |
⚡ Previous build results for gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-wpt are reusable. Rebuilding only android, mac-rel-css... |
⛄ The build was interrupted to prioritize another pull request. |
⚡ Previous build results for android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-wpt are reusable. Rebuilding only mac-rel-css... |
☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt |
fixes #8575
The implementation is naive, and doesn't support the
hardreload
parameter.And for the test, I'm not sure how else I can test the reload.