Bug 13635 - REGRESSION: Cannot log in to webmail.co.za account
Summary: REGRESSION: Cannot log in to webmail.co.za account
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac (PowerPC) OS X 10.4
: P1 Normal
Assignee: Nobody
URL: http://webmail.co.za/
Keywords: HasReduction, Regression
Depends on:
Blocks:
 
Reported: 2007-05-08 23:04 PDT by David Trevas
Modified: 2007-05-11 10:11 PDT (History)
1 user (show)

See Also:


Attachments
Reduction (600 bytes, text/html)
2007-05-09 09:02 PDT, mitz
no flags Details
Extended test case (875 bytes, text/html)
2007-05-09 10:14 PDT, mitz
no flags Details
Make HTMLOptionElement's text attribute match other browsers (22.40 KB, patch)
2007-05-11 03:20 PDT, mitz
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.