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

Handling of nested browsing contexts. #249

Closed
wants to merge 4 commits into from

Conversation

markafoltz
Copy link
Contributor

This PR updates the spec with the resolutions to Issue #79 (Presentations from within nested browsing contexts). It:

  • Patches the HTML5 spec to add an allow-presentation token to <iframe sandbox=>
  • Provides UI guidance that the UA should tell the user what origin is requesting presentation.

It also removes links to a couple of resolved issues.

@anssiko
Copy link
Member

anssiko commented Feb 1, 2016

Thanks for the pull! I committed some further clarifications to this branch:

  • Defined "presentation is disabled" we can reference elsewhere in the spec
  • Hooked that "presentation is disabled" into the entrypoint navigator.presentation (assuming the intended behaviour is to return null if presentation is disabled)
  • Made the informative text in the Sandboxing section a note for clarity

@mfoltzgoogle Feel free to merge this PR, if you're happy with the clarifications.

@markafoltz
Copy link
Contributor Author

Thanks for the edits @anssiko. Merging and closing issue.

@markafoltz markafoltz closed this Feb 1, 2016
@markafoltz markafoltz deleted the issue-79-nested-context branch May 26, 2016 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants