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

Issue 788910 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Proj-XR
Proj-XR-VR



Sign in to add a comment

VR: Go to DD Home rather than DOFF for 'X' button shown by WebVR timeout

Project Member Reported by mthiesse@chromium.org, Nov 27 2017

Issue description

When browsing is disabled and we detect that a WebVR page is failing to produce frames, we show some UX that provides and 'X' button for the user to exit presentation.

For CCTs/WebAPKs/Samsung devices that run into this case, we should make the 'X' button take the user to DD home as they cannot return to Browsing mode.
 
Labels: -M-64 M-65
This isn't important enough to merge back to 64.
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 16 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6a1d3c5ec4b916f3007a933812da0fd074f0ef3c

commit 6a1d3c5ec4b916f3007a933812da0fd074f0ef3c
Author: Michael Thiessen <mthiesse@chromium.org>
Date: Tue Jan 16 20:39:28 2018

VR: Return to DD home rather when exiting WebVR with browsing disabled

If VR browsing is disabled and the pages requests to exit WebVR, we
should return the user to DD home rather than asking them to take their
headset off.

Bug:  788910 
Change-Id: I0c5dc00af871c04744b8b0b27769281bb515b480
Reviewed-on: https://chromium-review.googlesource.com/868236
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: Yash Malik <ymalik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529498}
[modify] https://crrev.com/6a1d3c5ec4b916f3007a933812da0fd074f0ef3c/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellDelegate.java
[modify] https://crrev.com/6a1d3c5ec4b916f3007a933812da0fd074f0ef3c/chrome/browser/android/vr_shell/vr_shell_delegate.cc

Owner: mthiesse@chromium.org
Status: Fixed (was: Available)
Labels: Test-Manual
Labels: -Test-Manual Test-Complete
Status: Verified (was: Fixed)
Verified on build 66.0.3359.28.
Test case "VR fails, returns to DayDream Home" in "WebVR Manual Testing Plan"

Sign in to add a comment