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

Removing "creator browsing context" #2645

Closed
annevk opened this issue May 9, 2017 · 3 comments
Closed

Removing "creator browsing context" #2645

annevk opened this issue May 9, 2017 · 3 comments

Comments

@annevk
Copy link
Member

annevk commented May 9, 2017

In https://github.com/whatwg/html/tree/creator-document-wip I've stored some work in progress with the aim of making "creator browsing context" part of "creating a new browsing context". That seems like the right solution and will also help with solving #1445 eventually I think.

However, two bits are problematic at the moment:

It seems creator context security might be the wrong thing as https://w3c.github.io/webappsec-secure-contexts/ tries to obtain it from a document rather than a browsing context...

@mikewest
Copy link
Member

mikewest commented May 9, 2017

It seems creator context security might be the wrong thing as https://w3c.github.io/webappsec-secure-contexts/ tries to obtain it from a document rather than a browsing context...

I'm happy to do this in some other way. How would you like it to work?

@annevk
Copy link
Member Author

annevk commented May 9, 2017

@mikewest it seems you just want to propagate state to the created document, right? And probably also do that for non-initial about:blank and srcdoc and blob and such? I think we should just move this state to the document then and propagate it as appropriate.

@gsnedders
Copy link
Member

The current spec never really defines when the "creator browsing context" is passed to "create a new browsing context", and "creator browsing context" is never actually referenced outside of "create a new browsing context".

@annevk annevk closed this as completed in a48895e Jan 29, 2019
mustaqahmed pushed a commit to mustaqahmed/html that referenced this issue Feb 15, 2019
Some initial work to make introducing browsing context groups (see whatwg#4198) easier.

Fixes whatwg#2636 by making opener browsing context more generally applicable and making window.open() set it.

Fixes whatwg#2645 by removing creator browsing context and explicitly passing in the creator document.

Tests: web-platform-tests/wpt#15078.
mustaqahmed pushed a commit to mustaqahmed/html that referenced this issue Feb 15, 2019
Some initial work to make introducing browsing context groups (see whatwg#4198) easier.

Fixes whatwg#2636 by making opener browsing context more generally applicable and making window.open() set it.

Fixes whatwg#2645 by removing creator browsing context and explicitly passing in the creator document.

Tests: web-platform-tests/wpt#15078.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants