Bug 25682

Summary: [GTK] Expose password field with role of PASSWORD_TEXT
Product: WebKit Reporter: Joanmarie Diggs <jdiggs>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: VERIFIED FIXED    
Severity: Normal CC: apinheiro, walker.willie, xan.lopez
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on:    
Bug Blocks: 25531    
Attachments:
Description Flags
aforementioned test case
none
return PASSWORD_TEXT for password fields xan.lopez: review+

Joanmarie Diggs
Reported 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.
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+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joanmarie Diggs
Comment 1 2009-05-10 20:33:14 PDT
Created attachment 30170 [details] aforementioned test case
Jan Alonzo
Comment 2 2009-05-20 03:37:43 PDT
Created attachment 30504 [details] return PASSWORD_TEXT for password fields
Xan Lopez
Comment 3 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.
Jan Alonzo
Comment 4 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
Joanmarie Diggs
Comment 5 2009-05-20 14:37:22 PDT
Thanks Jan. Marking as VERIFIED.
Note You need to log in before you can comment on or make changes to this bug.