New issue
Advanced search Search tips

Issue 605377 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Chromevox does not remember last position on web page when switching windows

Reported by nimerjaber1@gmail.com, Apr 21 2016

Issue description

version: 52.0.2708.0
Reproduction Steps: 
1. eNABLE cHROMEVOX
2. Navigate to https://support.google.com/chromebook/answer/183101?hl=en&source=genius-rts
3. bEGIN READING THE WEB PAGE.
4. pRESS ALT+TAB TO SWITCH TO ANOTHER OPEN APPLICATION OR WINDOW.
5. pRESS ALT+TAB TO SWITCH BACK.
eXPECTED bEHAVIOR: cHROMEVOX WILL REMEMBER THE LAST PLACE YOU LEFT OFF.
aCTUAL BEHAVIOR: cHROMEVOX LOCATES ITSELF AT THE TOP OF THE PAGE.


 
Summary: Chromevox does not remember last position on web page when switching windows (was: cHROMEVOX DOES NOT REMEMBER LAST POSITION ON WEB PAGE WHEN SWITCHING WINDOWS)
Owner: dtseng@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 3 by dtseng@chromium.org, May 16 2016

Labels: -Type-Bug Type-Feature
This was a feature in Classic.
Labels: Phase3
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 12 2016

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

commit 7a99c7d3e7dc2ca71bbaa127099239355a41fd37
Author: dtseng <dtseng@chromium.org>
Date: Wed Oct 12 00:20:23 2016

Recovery: Implement focus recovery across root AutomationNodes

This cl maintains a WeakMap of AutomationRootNode objects to the last known node that had ChromeVox range.

This map gets updated manually when marked or implicitly when focus crosses top level root nodes.

TEST=manual; open context menu, open panel, activate menus that manipulate focus, press escape; ctrl+tab, alt+tab. Verify focus lands as expected.
BUG= 631918 , 628912 , 624586 , 605377 , 524673 , 652143 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/i_search.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js

Comment 6 by dtseng@chromium.org, Oct 12 2016

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
verified on 56.0.2903.0

Sign in to add a comment