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

Issue 821443 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Mar 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug
Proj-XR
Proj-XR-VR



Sign in to add a comment

VR: Chrome crashes when exiting VR while native dialog is open

Project Member Reported by tiborg@chromium.org, Mar 13 2018

Issue description

Chrome Version: 67.0.3368.0
OS: Android N

What steps will reproduce the problem?
(1) Go to https://permission.site
(2) Enter VR
(3) Click Camera
(4) While the permission dialog is open exit VR by pressing the X button

What is the expected result?
Cleanly exit VR

What happens instead?
Chrome crashes

Trace:
FATAL:VrShellImpl_jni.h(269)] Check failed: native. CloseAlertDialog

Stack Trace:
  RELADDR   FUNCTION                                                                       FILE:LINE
  000cd78d  logging::LogMessage::~LogMessage()                                             /usr/local/google/code/chromium/src/base/logging.cc:594:29
  00fdcabd  Java_org_chromium_chrome_browser_vr_1shell_VrShellImpl_nativeCloseAlertDialog  /usr/local/google/code/chromium/src/out/Debug/gen/chrome/browser/android/vr/vr_shell_jni_headers/vr_shell/jni/VrShellImpl_jni.h:269:3


 

Comment 1 by tiborg@chromium.org, Mar 14 2018

Labels: Hotlist-VRB-MVP
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 21 2018

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

commit 3937f71c1abac2f3c9fc65d4db6157e96e832b31
Author: Amirhossein Simjour <asimjour@chromium.org>
Date: Wed Mar 21 15:39:19 2018

Fix Chrome crash when exiting VR

Cancel dialogs before destroying their View.

Bug:  821443 
Change-Id: Ifb0eb47cc60efa3e0b7e6354754d5db0fef38cda
Reviewed-on: https://chromium-review.googlesource.com/969584
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Amirhossein Simjour <asimjour@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544712}
[modify] https://crrev.com/3937f71c1abac2f3c9fc65d4db6157e96e832b31/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java

Status: Fixed (was: Assigned)
Labels: Test-Complete
Status: Verified (was: Fixed)
Verified on build 67.0.3378.0, looks good.
We also have an automated test to cover this scenarios.
Issue 822159 has been merged into this issue.

Sign in to add a comment