Bug 13635

Summary: REGRESSION: Cannot log in to webmail.co.za account
Product: WebKit Reporter: David Trevas <trevas>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: mitz
Priority: P1 Keywords: HasReduction, Regression
Version: 523.x (Safari 3)   
Hardware: Mac (PowerPC)   
OS: OS X 10.4   
URL: http://webmail.co.za/
Attachments:
Description Flags
Reduction
none
Extended test case
none
Make HTMLOptionElement's text attribute match other browsers darin: review+

Description David Trevas 2007-05-08 23:04:14 PDT
I got a web-based e-mail account at webmail.co.za.

With WebKit Safari, clicking the Login button after entering my userid/passwd does nothing.

This works in the release version of Safari, as well as in Camino.
Comment 1 David Kilzer (:ddkilzer) 2007-05-09 07:38:08 PDT
If confirmed, this would be a regression.
Comment 2 David Kilzer (:ddkilzer) 2007-05-09 07:47:50 PDT
Test account details:

Username: webkit
Password: w3bk1t
City in which you were born: Cupertino

Comment 3 David Kilzer (:ddkilzer) 2007-05-09 08:26:58 PDT
In Firefox 2.0.0.3, Firebug reports:

document.floater has no properties in floater.js (line 16)
if(!mie){eval('document.'+chip.named+'.top ='+chip.yy);eval('document.'+chip.named+'.left='+chip.xx);}

Not sure if this error occurs in WebKit ToT as well, or if this even impacts logging in via WebKit ToT.  (Logging in via Firefox 2.0.0.3 works, however.)

Comment 4 mitz 2007-05-09 09:02:01 PDT
Created attachment 14432 [details]
Reduction

Firefox passes both cases, shipping WebKit fails the second case (leading space), TOT fails both cases (trailing space and leading space).
Comment 5 mitz 2007-05-09 10:14:55 PDT
Created attachment 14433 [details]
Extended test case

Firefox, WinIE and WebKit fail the third test, however WebKit actually displays the Yen symbol in the popup, so I think it should pass.
Comment 6 mitz 2007-05-09 11:59:46 PDT
It looks almost as if most of the logic can be moved from HTMLOptionElement::optionText() to HTMLOptionElement::text(), but the cases where the label attribute is non-empty seem tricky.
Comment 7 mitz 2007-05-11 03:20:27 PDT
Created attachment 14487 [details]
Make HTMLOptionElement's text attribute match other browsers
Comment 8 Darin Adler 2007-05-11 08:15:38 PDT
Comment on attachment 14487 [details]
Make HTMLOptionElement's text attribute match other browsers

r=me
Comment 9 Mark Rowe (bdash) 2007-05-11 10:11:30 PDT
Landed in r21410.