Bug 25682 - [GTK] Expose password field with role of PASSWORD_TEXT
Summary: [GTK] Expose password field with role of PASSWORD_TEXT
Status: VERIFIED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks: 25531
  Show dependency treegraph
 
Reported: 2009-05-10 20:32 PDT by Joanmarie Diggs
Modified: 2009-05-20 14:37 PDT (History)
3 users (show)

See Also:


Attachments
aforementioned test case (124 bytes, text/html)
2009-05-10 20:33 PDT, Joanmarie Diggs
no flags Details
return PASSWORD_TEXT for password fields (1.50 KB, patch)
2009-05-20 03:37 PDT, Jan Alonzo
xan.lopez: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joanmarie Diggs 2009-05-10 20:32:24 PDT
Steps to reproduce:

1. Open the (to be) attached test case.

2. Examine the accessible associated with the password field in Accerciser.

Expected results: The accessible's role would be PASSWORD_TEXT.

Actual results: The accessible's role is ENTRY.

Usage/Need: Screen readers such as Orca have an option to echo the keys being typed (i.e. to confirm to a user less familiar with the keyboard layout that each keypress is for the expected key). If this were to take place in a password field, the user would be announcing his/her password character by character each time he/she entered it. Therefore, it is necessary for screen readers to be able to distinguish traditional entries from password fields.
Comment 1 Joanmarie Diggs 2009-05-10 20:33:14 PDT
Created attachment 30170 [details]
aforementioned test case
Comment 2 Jan Alonzo 2009-05-20 03:37:43 PDT
Created attachment 30504 [details]
return PASSWORD_TEXT for password fields
Comment 3 Xan Lopez 2009-05-20 03:44:42 PDT
Comment on attachment 30504 [details]
return PASSWORD_TEXT for password fields

Looks good to me. One thing to consider is why there's no equivalent role in WebCore, and whether it might sense to add one.
Comment 4 Jan Alonzo 2009-05-20 04:19:34 PDT
(In reply to comment #3)
> (From update of attachment 30504 [details] [review])
> Looks good to me. One thing to consider is why there's no equivalent role in
> WebCore, and whether it might sense to add one.
> 

Landed in r43899
Comment 5 Joanmarie Diggs 2009-05-20 14:37:22 PDT
Thanks Jan. Marking as VERIFIED.