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

Support features in webxr #21159

Merged

Conversation

servo-wpt-sync
Copy link
Collaborator

Reviewed in servo/servo#25504.

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Servo project.

@Manishearth Manishearth changed the title WIP: support features in webxr Update xrSession_features_deviceSupport to use new user activations for each session request Jan 14, 2020
@Manishearth
Copy link
Contributor

cc @alcooper91

@servo-wpt-sync servo-wpt-sync changed the title Update xrSession_features_deviceSupport to use new user activations for each session request WIP: support features in webxr Jan 14, 2020
@servo-wpt-sync servo-wpt-sync changed the title WIP: support features in webxr Support features in webxr Jan 14, 2020
@Manishearth Manishearth changed the title Support features in webxr Update xrSession_features_deviceSupport to use new user activations for each session request Jan 14, 2020
@@ -31,34 +31,27 @@
}

return navigator.xr.test.simulateDeviceConnection(fakeDeviceInitParams)
.then((controller) => new Promise((resolve, reject) => {
navigator.xr.test.simulateUserActivation(() => {
.then((controller) => navigator.xr.test.simulateUserActivation(() =>
Copy link
Contributor

Choose a reason for hiding this comment

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

This last part here shouldn't be needed should it? You can just directly do the new method?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Probably yes

Copy link
Contributor

Choose a reason for hiding this comment

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

Fixed

@servo-wpt-sync servo-wpt-sync changed the title Update xrSession_features_deviceSupport to use new user activations for each session request Support features in webxr Jan 15, 2020
@servo-wpt-sync servo-wpt-sync force-pushed the servo_export_25504 branch 4 times, most recently from 458770b to 5c4b0c3 Compare January 21, 2020 23:48
@servo-wpt-sync servo-wpt-sync merged commit 5e8ddbc into web-platform-tests:master Jan 22, 2020
@servo-wpt-sync servo-wpt-sync deleted the servo_export_25504 branch January 22, 2020 05:50
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

5 participants