New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 673912 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Proj-VR
Proj-XR



Sign in to add a comment

Skipping DON flow fires vrdisplayactivate event an extra time

Project Member Reported by bsheedy@chromium.org, Dec 13 2016

Issue description

Skipping the DON flow via the Daydream developer options causes WebVR's vrdisplayactivate event to fire an extra time. This results in the following behavior:

- requestPresent called manually, DON flow completed: no event fired
- requestPresent called manually, DON flow skipped: one event fired
- NFC scanned, DON flow completed: one event fired
- NFC scanned, DON flow skipped: two events fired

This doesn't affect our ability to test, as we can wait for a certain number of events to fire before continuing. However, it would be preferable for DON flow completion and skipping to have the same behavior. I'm not sure if this is a bug with Chrome or with Daydream - if it turns out to be the latter, we can pass this off to them.

This can be reproduced by visiting https://webvr.info/samples/03-vr-presentation.html , inspecting the tab with developer tools, and running

window.addEventListener('vrdisplayactivate', () => {console.error("vrdisplayactivate fired");}, false);

in the console. Then, entering VR with or without DON flow skipping enabled will yield different numbers of console logs.
 
Some further investigation. When DON flow skipped, in NFC scanned scene,
VrShellDelegate.java: enterVRFromIntent will be called twice, with intent content {com.google.intent.category.DAYDREAM},
But when DON flow completed, it will be called only once.
Components: -UI>Browser>VR Blink>WebVR
Labels: -Proj-VR Proj-VR-Daydream

Comment 3 by sko...@chromium.org, Jan 12 2017

Cc: bshe@chromium.org
Status: Available (was: Untriaged)
Owner: bsheedy@chromium.org
Status: Assigned (was: Available)
Is this fixed Brian? If so please close, otherwise assign it to me and I'll fix it.
Status: WontFix (was: Assigned)
Seems to be fixed, closing.
Components: Blink>WebXR

Sign in to add a comment