New issue
Advanced search Search tips

Issue 781389 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Proj-XR



Sign in to add a comment

Several VR tests failing on N+

Project Member Reported by bsheedy@chromium.org, Nov 3 2017

Issue description

The following tests have started failing on N and O, although this is likely due to being run with Daydream View paired instead of Cardboard:

org.chromium.chrome.browser.vr_shell.VrShellControllerInputTest#testAppButtonExitsFullscreen
org.chromium.chrome.browser.vr_shell.VrShellNavigationTest#test2dFullscreenToWebVr
org.chromium.chrome.browser.vr_shell.VrShellNavigationTest#testWebVrFullscreenTo2d
org.chromium.chrome.browser.vr_shell.VrShellNavigationTest#testWebVrFullscreenToWebVr
org.chromium.chrome.browser.vr_shell.VrShellTransitionTest#testExitFullscreenAfterExitingVrFromCinemaMode

This is being caused by timing out waiting for JavaScript portions of the tests. example https://logs.chromium.org/v/?s=chromium%2Fbb%2Fchromium.fyi%2FAndroid_VR_Tests%2F13370%2F%2B%2Frecipes%2Fsteps%2Fchrome_public_test_vr_apk__2_%2F0%2Fsteps%2Fchrome_public_test_vr_apk.chrome_public_test_vr_apk__2_%2F0%2Fstdout

This is occurring on both our FYI bot and the Nougat Phone Tester bot on chromium.android.
 
Owner: mdjones@chromium.org
Status: Assigned (was: Untriaged)
The cause has been tracked to https://chromium-review.googlesource.com/c/chromium/src/+/736049

Comment 2 Deleted

Issue 782214 has been merged into this issue.
Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 10 2017

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

commit 8a62d5606144d44eb304d860e4e5a3f0764815db
Author: bsheedy <bsheedy@chromium.org>
Date: Fri Nov 10 18:42:31 2017

Fix failing VR instrumentation tests

Fixes a few VR tests that were failing because Chrome thought the tabs
were not interactable.

Bug:  781389 
Change-Id: Ib6b0a8f77625356cd08a6f03ba8e1248f2306de5
Reviewed-on: https://chromium-review.googlesource.com/761439
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515610}
[modify] https://crrev.com/8a62d5606144d44eb304d860e4e5a3f0764815db/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java

Status: Fixed (was: Started)
Labels: M-64 Test-Complete
Labels: VR-Caught-By-Test
Components: Internals>XR
Labels: -VR-Caught-By-Test XR-Caught-By-Test

Sign in to add a comment