diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2017-02-16 16:57:05 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-02-16 16:57:05 -0800 |
commit | 6adbcb4ccdc1f74638b0c6e990c122e34bc967e4 (patch) | |
tree | aaf4d3cf5de85ea3dfabffe6736f30d2f278fcf0 /python/tidy/servo_tidy_tests | |
parent | 11396b4dd3834d6794bd4e32f30c1df96fc6a01d (diff) | |
parent | e8d765557f2ef1a298eed6d4234e74ba45333a44 (diff) | |
download | servo-6adbcb4ccdc1f74638b0c6e990c122e34bc967e4.tar.gz servo-6adbcb4ccdc1f74638b0c6e990c122e34bc967e4.zip |
Auto merge of #15358 - asajeffrey:script-browsingcontext-xorigin-window, r=jdm
Allow browsing contexts to resolve to cross-origin windows
<!-- Please describe your changes on the following line: -->
This PR implements cross-thread `WindowProxy` objects.
At the moment, if a `Window` performs a non-similar-origin navigation, the old script thread does not update its `WindowProxy`, since the new `Window` is in the new script thread. With this PR, the `WindowProxy` is updated to a dummy `XOriginWindow` object, that only implements the whitelisted methods that are allowed to be called cross-origin.
This PR does not include working implementations of some of the cross-origin `Window` or `Location` methods.
This PR causes some cross-origin wpt tests to now pass, in particular `/html/browsers/origin/cross-origin-objects/cross-origin-objects.html ` now passes `Only whitelisted properties are accessible cross-origin`. There are some CORS failures in `fetch`, I suspect caused by the incorrect setting of `origin` in fetch requests.
Although there are some functions that now throw `SecurityException`, it is not meant to be a complete implementation, which will have to wait for XOWs to land.
---
<!-- 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 #15180.
- [X] There are tests for these changes
<!-- 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/15358)
<!-- Reviewable:end -->
Diffstat (limited to 'python/tidy/servo_tidy_tests')
0 files changed, 0 insertions, 0 deletions