New issue
Advanced search Search tips

Issue 914684 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 18
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug
Team-Accessibility



Sign in to add a comment

Unable to turn on ChromeVox while full-screen ARC++ app is on screen

Project Member Reported by leberly@chromium.org, Dec 13

Issue description

Google Chrome	72.0.3626.8 (Official Build) dev (64-bit) 
Firmware Version	Google_Eve.9584.174.0 


Steps to repro:
# Open the Play Store and install any full-screen app, I used Steam.
# Open Steam in full screen mode
# Press ctrl + alt + z
Expected: ChromeVox turns on
Actual: nothing happens 
 
Owner: yawano@chromium.org
Status: Assigned (was: Available)
Yuki, could you take a look?

Cc: yhanada@chromium.org
Yes, I'll take a look. It looks that this happens if an Android app consumes the key event. I tried with Youtube app and this hasn't happened.
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 18

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

commit 3da77ca73813b9a2699ff1135c869f9f94681603
Author: Yuki Awano <yawano@chromium.org>
Date: Tue Dec 18 23:44:08 2018

Add ChromeVox activation key combo as reserved key combo

- Android app may consume ChromeVox activation key combo. Add the key
  combo (Ctrl+Alt+Z) as a Chrome OS reserved key combo.

Bug: b/121100170
Bug:  914684 
Test: Tested as described in crbug.
Change-Id: I9ad69ef726a09fb0d18defc88357f7c696a0d7f8
Reviewed-on: https://chromium-review.googlesource.com/c/1381322
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Yuki Awano <yawano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#617674}
[modify] https://crrev.com/3da77ca73813b9a2699ff1135c869f9f94681603/components/exo/keyboard.cc

Status: Fixed (was: Assigned)
Cc: lpalmaro@chromium.org yawano@chromium.org
Labels: M-73
Owner: lpalmaro@chromium.org
lpalmaro@: Let me know if we want to merge this for M72.

Sign in to add a comment