New issue
Advanced search Search tips

Issue 711483 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug
Team-Accessibility



Sign in to add a comment

Crash pressing overview mode button in overview mode

Project Member Reported by kathrelk...@chromium.org, Apr 13 2017

Issue description

Chrome Version: 9455.0.0, 59.0.3068.1, Caroline and Samus

What steps will reproduce the problem?
(1) Press overview button and enter overview mode.
(2) Press button again.

Crash ID: 2e02360e80000000


Stack trace points to NotifyAccessibilityEvent
 
Cc: varkha@chromium.org
Labels: ReleaseBlock-Stable
adding RBS since we really shouldn't ship to stable users with this
Cc: zork@chromium.org wutao@chromium.org
Is this a recent regression?
Status: Available (was: Untriaged)
Steps that reproduce this for me:
(1) Enable Accessibility > ChromeVox
(2) Press overview button and enter overview mode.
(3) Press button again.

Comment 4 by zork@chromium.org, May 5 2017

Owner: zork@chromium.org

Comment 5 by zork@chromium.org, May 5 2017

Cc: dmazz...@chromium.org
We're seeing this without ChromeVox enabled, on a newly added account.

Minnie (59.0.3071.30):
crash/4393023e80000000
http://feedback/#/Report/59475859846

Comment 7 by wutao@chromium.org, May 6 2017

I can repeat #3 at Minnie (59.0.3071.30).
And after exiting ChromeVox mode, I also can repeat #6, but I cannot repeat #6 without enable ChromeVox first.
Is it possible some status is not cleared after exiting ChromeVox mode?

Issue 718135 could be related to this:
https://bugs.chromium.org/p/chromium/issues/detail?id=718135


Mergedinto: 718135
Status: Duplicate (was: Available)
Duplicate of 718135

Project Member

Comment 9 by bugdroid1@chromium.org, May 9 2017

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

commit 546a3b42a3414c671362459b1519eaeefb43afc3
Author: zork <zork@chromium.org>
Date: Tue May 09 06:36:43 2017

Check that a view exists before sending an accessibility event.

When handling bounds changes, the view can disappear before the
AXWindowObjectWrapper gets the event.

BUG= 711483 
TEST=Turn on accessibility, and press the overview button twice.
Ensure it does not crash.

Review-Url: https://codereview.chromium.org/2865923004
Cr-Commit-Position: refs/heads/master@{#470249}

[modify] https://crrev.com/546a3b42a3414c671362459b1519eaeefb43afc3/ui/views/accessibility/ax_window_obj_wrapper.cc

Project Member

Comment 10 by bugdroid1@chromium.org, May 11 2017

Labels: merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce6a3cb8deba99f5c5ce2fcf8b20e0260aa166a3

commit ce6a3cb8deba99f5c5ce2fcf8b20e0260aa166a3
Author: zork@chromium.org <zork@google.com>
Date: Thu May 11 18:38:46 2017

Check that a view exists before sending an accessibility event.

When handling bounds changes, the view can disappear before the
AXWindowObjectWrapper gets the event.

BUG= 711483 
TEST=Turn on accessibility, and press the overview button twice.
Ensure it does not crash.

Review-Url: https://codereview.chromium.org/2865923004
Cr-Original-Commit-Position: refs/heads/master@{#470249}
Review-Url: https://codereview.chromium.org/2878733003 .
Cr-Commit-Position: refs/branch-heads/3071@{#517}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/ce6a3cb8deba99f5c5ce2fcf8b20e0260aa166a3/ui/views/accessibility/ax_window_obj_wrapper.cc

Sign in to add a comment