New issue
Advanced search Search tips

Issue 692380 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

MacViews/a11y: Text-specific a11y attributes should not be supported for protected views::Textfields.

Project Member Reported by patricia...@chromium.org, Feb 15 2017

Issue description

NSSecureTextfields don't support any of the text-specific accessibility attributes as listed in  Issue 678898 . Chrome with MacViews currently does, but we should match Cocoa's behaviour.

To repro:

1. Open the Accessibility Inspector.
2. Open the HTTP Auth dialog in Chrome.
3. Use the Accessibility Inspector to inspect the password field. It should not show any of the text-specific attributes as listed in the bug mentioned previously, e.g 'SelectedText', 'SelectedTextRange', 'NumberOfCharacters', etc.
 
Status: Assigned (was: Unconfirmed)
Status: Started (was: Assigned)
https://codereview.chromium.org/2706743002/
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 28 2017

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

commit 54b3acd0692e87dc4d33aa2ffb66ae7697a16f79
Author: patricialor <patricialor@chromium.org>
Date: Tue Feb 28 00:39:49 2017

MacViews/a11y: Disable most text-specific attributes for protected textfields.

Cocoa doesn't support these attributes in NSSecureTextfields, so match that with
MacViews.

BUG= 692380 

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

[modify] https://crrev.com/54b3acd0692e87dc4d33aa2ffb66ae7697a16f79/base/mac/sdk_forward_declarations.h
[modify] https://crrev.com/54b3acd0692e87dc4d33aa2ffb66ae7697a16f79/ui/accessibility/platform/ax_platform_node_mac.mm
[modify] https://crrev.com/54b3acd0692e87dc4d33aa2ffb66ae7697a16f79/ui/views/widget/native_widget_mac_accessibility_unittest.mm

Status: Fixed (was: Started)

Sign in to add a comment