Skip to content
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

Request.clone/Response.clone do not specify in which global the new object belongs to. #825

Open
bakulf opened this issue Nov 1, 2018 · 3 comments

Comments

@bakulf
Copy link

bakulf commented Nov 1, 2018

In which global should be created the new request? Same question for a cloned Response object.
Currently, in Firefox we use the global of the original request/response object.

@jakearchibald
Copy link
Collaborator

Good catch. I agree with what Firefox is doing.

@annevk
Copy link
Member

annevk commented Nov 1, 2018

Yeah me too, that way any prototype expandos also remain available etc. I'll see about testing this at least. I think the behavior would fall out of fixing whatwg/webidl#135 in IDL.

annevk added a commit to web-platform-tests/wpt that referenced this issue Nov 1, 2018
@annevk
Copy link
Member

annevk commented Nov 1, 2018

I created web-platform-tests/wpt#13850 to test this. As far as I can tell all browsers agree here.

annevk added a commit to web-platform-tests/wpt that referenced this issue Nov 2, 2018
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Nov 10, 2018
…only

Automatic update from web-platform-testsRequest/Response's clone() realm

For whatwg/fetch#825.
--

wpt-commits: e5ba0a871d891aabfe286935d31cbae6b4a850c3
wpt-pr: 13850
jyc pushed a commit to jyc/gecko that referenced this issue Nov 11, 2018
…only

Automatic update from web-platform-testsRequest/Response's clone() realm

For whatwg/fetch#825.
--

wpt-commits: e5ba0a871d891aabfe286935d31cbae6b4a850c3
wpt-pr: 13850
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 3, 2019
…only

Automatic update from web-platform-testsRequest/Response's clone() realm

For whatwg/fetch#825.
--

wpt-commits: e5ba0a871d891aabfe286935d31cbae6b4a850c3
wpt-pr: 13850

UltraBlame original commit: eed10c92b6a157db3ce3d9ec5ea7361c4b0fdf47
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 3, 2019
…only

Automatic update from web-platform-testsRequest/Response's clone() realm

For whatwg/fetch#825.
--

wpt-commits: e5ba0a871d891aabfe286935d31cbae6b4a850c3
wpt-pr: 13850

UltraBlame original commit: eed10c92b6a157db3ce3d9ec5ea7361c4b0fdf47
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 3, 2019
…only

Automatic update from web-platform-testsRequest/Response's clone() realm

For whatwg/fetch#825.
--

wpt-commits: e5ba0a871d891aabfe286935d31cbae6b4a850c3
wpt-pr: 13850

UltraBlame original commit: eed10c92b6a157db3ce3d9ec5ea7361c4b0fdf47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants