977592 - HTTP auth dialog puts focus in password field by default

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

As of not too long ago (sometime in the last month, I think) I've noticed that when I go to pages which force a HTTP auth dialog, the focus is in the password field rather than the username field by default. At least on my Galaxy Q running 2.3.

STR:
1. Go to a page with HTTP auth (e.g. https://staktrace.com/apps/bugmash/dashboard.php)
2. Wait for the HTTP auth dialog to pop up

Expected: focus is in the username field
Actual: focus is in the password field

Comment 1

[Aaron Train [:aaronmt]]

I see this too on my Nexus 4 (4.4.2).

Last good revision: 660b62608951 (2014-02-20)
First bad revision: 7010ab83a06e (2014-02-21)
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=660b62608951&tochange=7010ab83a06e


a10bb6a9da93	Mark Capella — Bug 927882 - Standardize TextSelection caret positioning logic for soft keyboards, r=jchen, wesj, f=margaret

Comment 2

[Aaron Train [:aaronmt]]

547342061dc3	Wes Johnston — Bug 931900 - Make null value parameter handling match Desktop. r=margaret

Comment 3

[Mark Capella [:capella]]

I can repro the situation as stated, but I can't think of any way my patch(s) would affect positioning of the blinking IME caret's field position, (vs positioning of the orange TextSelection UI caret).

(Fyi the issue doesn't seem to exist on desktop).

For good measure, I backed out my changes and re-tested and the issue still exists.

Comment 4

[Aaron Train [:aaronmt]]

Yes, why I cancelled your need-info.

Comment 5

[Wesley Johnston (:wesj)]

Attachment: Patch

I just flipped this when we recently changed things.

Comment 6

[Wesley Johnston (:wesj)]

http://hg.mozilla.org/integration/fx-team/rev/9f335dcd5db1

Comment 7

[(PTO June 19-July 7) Ryan VanderMeulen]

https://hg.mozilla.org/mozilla-central/rev/9f335dcd5db1