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

Presentation API: Sandboxing test on PresentationRequest constructor #4542

Merged

Conversation

tidoust
Copy link
Contributor

@tidoust tidoust commented Jan 16, 2017

User agents show now check sandboxing flags when PresentationRequest objects are constructed.

This update also drops sandboxing tests that were checking failures to call "start", "reconnect", and "getAvailability" in the absence of the "allow-presentation" sandbox attribute, because there should be no longer any way to reach these steps. Such scripts will simply always fail to create PresentationRequest objects.

User agents show now check sandboxing flags when PresentationRequest objects
are constructed.

This update also drops sandboxing tests that were checking failures to call
"start", "reconnect", and "getAvailability" in the absence of the
"allow-presentation" sandbox attribute, because there should be no longer
any way to reach these steps. Such scripts will simply always fail to create
PresentationRequest objects.
@wpt-pr-bot
Copy link
Collaborator

Notifying @louaybassbouss, @tidoust, and @zqzhang. (Learn how reviewing works.)

@tidoust
Copy link
Contributor Author

tidoust commented Jan 31, 2017

@tomoyukilabs or @louaybassbouss Could you review this pull request?

Copy link
Member

@tomoyukilabs tomoyukilabs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tidoust This looks good to me.

@tidoust tidoust merged commit 45e9a84 into web-platform-tests:master Feb 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants