New issue
Advanced search Search tips

Issue 619417 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Jul 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Notification permissions is silent

Project Member Reported by chaok@google.com, Jun 12 2016

Issue description

Mode: force_next
Version: 52.0.2743.32
Reproduction Steps: 
1. drive/
2. turn on notifications
3. Attempt to allow/deny

Observed:
permission dialog is silent

Expected:
permission dialog to be accessible to cvox2 (Next)

 
Project Member

Comment 1 by sheriffbot@chromium.org, Jun 13 2016

Labels: Hotlist-Google
Labels: Phase3
Owner: dtseng@chromium.org
Status: Available (was: Unconfirmed)
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 15 2016

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

commit 4d356e830cd6c653cf7979e16754faeef1e6f953
Author: dtseng <dtseng@chromium.org>
Date: Wed Jun 15 23:42:03 2016

Fix alert dialog output.

An alert dialog uses $descendants for its enter rule, but this potentially results in infinite recursion since a descendant's format rule may walk up to the alert dialog to format an ancestor. We should never use children in the formatting of enter rules.

A specific case where this hits is in the
google.com -> search by voice

permissions dialog. Moving forward from the close button results in the above mentioned infinite recursion.

BUG= 619417 , 619388 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/4d356e830cd6c653cf7979e16754faeef1e6f953/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js

Comment 4 by chaok@google.com, Jul 8 2016

Microphone permissions when doing Voice Search on GOogle.com works with ChromeVox Next:
54.0.2787.0
Status: fixed (was: Available)
Status: Verified (was: Fixed)
As per #4

Sign in to add a comment