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

Issue 910325 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Chromevox cannot be activated twice on smart displays

Project Member Reported by rmrossi@chromium.org, Nov 29

Issue description

OS: Chromecast

What steps will reproduce the problem?
(1) Enable screen reader
(2) Disable screen reader
(3) Enable screen reader

What is the expected result?
Should be able to touch explore and use screen reader.

What happens instead?
Nothing is ever foucsed.  Screen reader does not function at all.




 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 30

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

commit ba36bdbbbced29c4bf218766d2c86bd0ec569a9f
Author: Randy Rossi <rmrossi@chromium.org>
Date: Fri Nov 30 14:05:55 2018

Fix screen reader on chromecast not activating twice

This CL makes sure the bounds on the serialized node created
in ax_root_obj_wrapper.cc are set. Previously, this was left
as 0,0 and due to recent condition added to ui_event_generator.cc,
this was preventing auto generated LOAD_COMPLETE event from
firing on the desktop tree. (See ShouldFireEvent function).

Also removed all #if !defined(IS_CHROMECAST) as they had no
    effect anyway (the define was never set). No reason to omit
    this code for chromecast anyway.

Change-Id: Ifff95227a8b07349e5401678f9428a85b803ca01

      screen reader.  No issues detected.  Did the same on
      Chromecast smart display. Same.

Bug: 910325
Test: Manual install on Samus ChromeOS device, activated
Change-Id: Ifff95227a8b07349e5401678f9428a85b803ca01
Reviewed-on: https://chromium-review.googlesource.com/c/1355551
Reviewed-by: David Tseng <dtseng@chromium.org>
Commit-Queue: Randy Rossi <rmrossi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#612636}
[modify] https://crrev.com/ba36bdbbbced29c4bf218766d2c86bd0ec569a9f/ui/views/accessibility/ax_root_obj_wrapper.cc

Components: UI>Accessibility>ChromeVox

Sign in to add a comment