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

Issue 881970 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug
Proj-VR
Proj-XR
Proj-XR-VR



Sign in to add a comment

VR: Spinner doesn't appear when WebXR takes too long.

Project Member Reported by acondor@chromium.org, Sep 7

Issue description

Chrome Version: 69.0.3497.76
OS: Android

What steps will reproduce the problem?
(1) Enable WebVR in chrome://flags
(2) Visit: https://webvr.info/samples/test-slow-render.html?noFrames=1
(3) Click "Enter VR"

This only happens in Cardboard, but not in Daydream.

What is the expected result?

After a couple of seconds, a spinner appears.

What happens instead?

Screen remains black

It works in: 68.0.34440.91

 
Description: Show this description
Status: Started (was: Available)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 10

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

commit fadad4d67ab25610e0d14143fd2996d85c9d3981
Author: Aldo Culquicondor <acondor@chromium.org>
Date: Mon Sep 10 16:01:59 2018

VR: Fix spinner for slow WebXR rendering

Bug:  881970 
Change-Id: I23d7c563929d0c95459050669b3303eba3785128
Reviewed-on: https://chromium-review.googlesource.com/1216464
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Aldo Culquicondor <acondor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#589930}
[modify] https://crrev.com/fadad4d67ab25610e0d14143fd2996d85c9d3981/chrome/browser/android/vr/gvr_scheduler_delegate.cc

Comment 4 Deleted

 Issue 891539  has been merged into this issue.
Labels: Merge-Request-70
Status: Started (was: Fixed)
Project Member

Comment 7 by sheriffbot@chromium.org, Oct 15

Labels: -Merge-Request-70 Merge-Review-70 Hotlist-Merge-Review
This bug requires manual review: We are only 0 days from stable.
Please contact the milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), geohsu@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
This is probably a nice to have for 70. Let me know if you prefer to disregard.
Labels: -Hotlist-Merge-Review -Merge-Review-70
As this is already in the stable release, I think it's fine to just target 71.

Feel free to mark as fixed as this already landed before 71 branched.
Status: Fixed (was: Started)

Sign in to add a comment