New issue
Advanced search Search tips

Issue 641728 link

Starred by 4 users

Issue metadata

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



Sign in to add a comment

Selecting/copying text will repeat previous content on clipboard or not copy

Project Member Reported by chaok@google.com, Aug 28 2016

Issue description

Mode: force_next
Version: 54.0.2837.0
Reproduction Steps: 
1. Linear-nav to text to copy
2. search+s to start/end selection  (try object, line, word, or chars)
3. ctrl+c to copy  > ctrl=v to paste

Observed: Previously content of clipboard is uttered and nothing is copied
Expected: selected text to be copied and uttered
 
I am not seeing this... where is this occurring? 

Comment 2 by chaok@google.com, Aug 28 2016

Mostly when copying text from inbox
Project Member

Comment 3 by sheriffbot@chromium.org, Aug 28 2016

Labels: Hotlist-Google
Labels: Phase3 inbox
Owner: dtseng@chromium.org
Status: Assigned (was: Unconfirmed)
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/+/a7d8eae7194d623c7e2ff753f79524da9864af7e

commit a7d8eae7194d623c7e2ff753f79524da9864af7e
Author: dtseng <dtseng@chromium.org>
Date: Wed Oct 12 17:41:58 2016

Make ChromeVox use child-index based offsets again for selection.

Make further adjustments to selection node and index:
- line breaks, when selected, appear to do nothing in Blink/AX; this is either passing offsets before or after the line break. Make the index adjument walk up to the child of the selected node and use its index in parent
- adjust nodes accordingly as well
- the clipboard text output requires the clipboard actually have the copied text. Since ChromeVox appears to get the ctrl+c first, this is often stale.

BUG= 642000 , 641728 
TEST=manual
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/a7d8eae7194d623c7e2ff753f79524da9864af7e/chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js
[modify] https://crrev.com/a7d8eae7194d623c7e2ff753f79524da9864af7e/chrome/browser/resources/chromeos/chromevox/cvox2/background/cursors.js
[modify] https://crrev.com/a7d8eae7194d623c7e2ff753f79524da9864af7e/chrome/browser/resources/chromeos/chromevox/cvox2/background/cursors_test.extjs

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

Status: fixed (was: Assigned)

Comment 7 by chaok@google.com, Oct 19 2016

Status: Verified (was: Fixed)
Fixed in: 56.0.2894.0

Sign in to add a comment