New issue
Advanced search Search tips

Issue 647092 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Password fields should utter "secured" and "# characters" in field

Project Member Reported by chaok@google.com, Sep 15 2016

Issue description

Mode: force_next
Version: 55.0.2858.0
Reproduction Steps: 
1. Go to *any* password field
2. Type in password
3. TAB/Shift+tab to re-focus field

Observed: "password, edit text"
Expected: Password, secured edit text, # characters"



 
Project Member

Comment 1 by sheriffbot@chromium.org, Sep 15 2016

Labels: Hotlist-Google
Labels: Phase3
Owner: dtseng@chromium.org
Status: Assigned (was: Untriaged)

Comment 3 by dtseng@chromium.org, Sep 30 2016

Expected: Password, secured edit text, # characters"

I agree with the count announcement, but password and secure are redundant. Change pending for the count output.
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 30 2016

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

commit 5723dd766de19a0acbc14b740d702ae71de61485
Author: dtseng <dtseng@chromium.org>
Date: Fri Sep 30 22:50:16 2016

Fix reading of password values.

Chrome exposes passwords using bullets (\u2022). Add this to the repetition regex which gets used to replace multiple occurances with an internationalized formatted string like
1 dot
2 dots

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
BUG= 647092 

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

[modify] https://crrev.com/5723dd766de19a0acbc14b740d702ae71de61485/chrome/browser/resources/chromeos/chromevox/common/editable_text_base.js
[modify] https://crrev.com/5723dd766de19a0acbc14b740d702ae71de61485/chrome/browser/resources/chromeos/chromevox/host/interface/abstract_tts.js
[modify] https://crrev.com/5723dd766de19a0acbc14b740d702ae71de61485/chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings.grd

Comment 5 by dtseng@chromium.org, Sep 30 2016

Status: Fixed (was: Assigned)

Comment 6 by chaok@google.com, Oct 2 2016

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

Sign in to add a comment