Bug 6221

Summary: REGRESSION: Password field is blank error for secure websites using password field (ToT, 12/24/05) .
Product: WebKit Reporter: Rosyna <webkit-bugs>
Component: FormsAssignee: Geoffrey Garen <ggaren>
Status: RESOLVED FIXED    
Severity: Blocker    
Priority: P1    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: https://www.bankone.com/bank/BolLogin.aspx
Attachments:
Description Flags
Change all slotTextChanged()s to take a DOMString
mjs: review+
ChangeLog entry to go with the patch none

Rosyna
Reported 2005-12-23 14:42:00 PST
No matter what password I put into this form, I get a javascript error that the password field is blank. This also happens with http://bugreport.apple.com it fails every time I attempt to login.
Attachments
Change all slotTextChanged()s to take a DOMString (4.54 KB, patch)
2005-12-24 01:01 PST, mitz 		mjs: review+
DetailsFormatted DiffDiff
ChangeLog entry to go with the patch (696 bytes, text/plain)
2005-12-24 01:23 PST, mitz 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Maciej Stachowiak
Comment 1 2005-12-23 22:06:31 PST
I think all form fields are currently broken, both text and password. This seems to be a very recent regression.
mitz
Comment 2 2005-12-24 01:01:22 PST
Created attachment 5257 [details] Change all slotTextChanged()s to take a DOMString
Maciej Stachowiak
Comment 3 2005-12-24 01:15:32 PST
Great fix. It is a bit disturbing that all text fields in forms completely broke and layout tests did not catch it. Unfortunately, it looks like in this case only typing in text fields was broken, not programmatic changes, so it's not possible to make an automated test case for this right now.
mitz
Comment 4 2005-12-24 01:23:31 PST
Created attachment 5258 [details] ChangeLog entry to go with the patch
Eric Seidel (no email)
Comment 5 2005-12-24 02:33:56 PST
My appologies. A shame there is no way to test this.
Joost de Valk (AlthA)
Comment 6 2006-01-22 04:56:08 PST
Removing keyword(s) since bug is fixed.
Joost de Valk (AlthA)
Comment 7 2006-01-22 05:00:23 PST
Removing keyword(s) since bug is fixed.
Eric Seidel (no email)
Comment 8 2006-01-31 21:20:38 PST
Removing Regression keyword from bugs already fixed.
Note You need to log in before you can comment on or make changes to this bug.