Bug 7405 – REGRESSION: password field should have focus after loading webmail.mac.com

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Vicki Murley]

This bug is also in Radar as <rdar://4422657>

Go to webmail.mac.com.  After the page loads, the password field on this page should have focus.

Works in Safari on 10.4.4.

-------------------------------------------

<GMT26-Jan-2006 07:45:11GMT> Vicki Murley:
Crash reported in 4422624.

<GMT26-Jan-2006 19:32:45GMT> Adele Peterson:
Our new focus() code uses isFocusable() to determine whether or not to try to focus.  At the load time, there isn't a renderer for the input element, so isFocusable() fails.  Luckily, this works fine with our new text fields- so this problem should go away when we switch over.

<GMT26-Jan-2006 19:50:55GMT> Vicki Murley:
One important thing to know when testing this fix -
If i load the page, see it fail, close the window, load the page again -- it works.  The password field has focus.  This only fails the first time you launch Safari (maybe also after waiting a bit of time btwn tests).  

<GMT26-Jan-2006 22:53:39GMT> Adele Peterson:
I realized that my earlier test was wrong.  I think the reason there isn't a renderer is that the style sheet hasn't loaded yet.

<GMT26-Jan-2006 22:54:42GMT> Adele Peterson:
We used to not care about the renderer, since we'd just set the focus node, and then when the element's widget got focus, it would run some code to reveal the element.

[Adele Peterson]

I have a fix for this.

[Adele Peterson]

Committed revision 15532