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

Issue 747452 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 763143
Owner:
Last visit > 30 days ago
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

HTML lists cannot be selected as a whole via accessibility API

Project Member Reported by nek...@chromium.org, Jul 21 2017

Issue description

Snippet:
<ul>
<li>item1</li>
<li>item2</li>
</ul>
Steps to reproduce:
1. Start Jaws 18 and load the above snippet.
2. Focus the beginning of the list in the virtual buffer by pressing Ctrl+Home.
3. Press F8 to select the whole list. (F8 is a Jaws shortcut for selecting the whole object under the virtual cursor.)
4. Press Ctrl+C to copy.
5. Notice that nothing has been copied.

Cause:
Calling AXLayoutObject::SetSelection with one of the selection endpoints set to a list marker fails. The list marker in this case is at the anchor of the selection, i.e. the bullet before the first list item.


 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 21 2017

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

commit 21937dad6f25f94936fc2e34a82b2577dfc11128
Author: Nektarios Paisios <nektar@chromium.org>
Date: Fri Jul 21 18:47:30 2017

Blink should be able to handle selection on AX objects with no nodes.

Tries to deal with the most egregious bugs reported by VFO that have to do with selection.
R=dmazzoni@chromium.org
TESTED=Layout tests, manually with Jaws

Bug:  747452 
Change-Id: I5806087e4c9871150fc33dd423641d61fadee1e3
Reviewed-on: https://chromium-review.googlesource.com/580541
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488721}
[modify] https://crrev.com/21937dad6f25f94936fc2e34a82b2577dfc11128/content/browser/accessibility/browser_accessibility_com_win.cc
[add] https://crrev.com/21937dad6f25f94936fc2e34a82b2577dfc11128/third_party/WebKit/LayoutTests/accessibility/list-with-selection.html
[modify] https://crrev.com/21937dad6f25f94936fc2e34a82b2577dfc11128/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp

Comment 2 by nek...@chromium.org, Dec 10 2017

Mergedinto: 763143
Status: Duplicate (was: Started)

Sign in to add a comment