Closed
Bug 900007
Opened 11 years ago
Closed 11 years ago
Don't work arrow key on location bar when using autocomplete address
Categories
(Firefox :: Address Bar, defect)
Tracking
()
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
firefox24 | --- | unaffected |
firefox25 | + | verified |
People
(Reporter: tetsuharu, Assigned: ehsan.akhgari)
References
Details
(Keywords: regression)
[Environment] * http://hg.mozilla.org/mozilla-central/rev/c2b375f3a909 * OS X 10.8 [Step To reproduce] 1. Focus location bar. 2. Type some words to lookup history or bookmarks. 3. Display drop down list of matched entries. 4. Input up/down key. [Result] * Drop down list will flash. (Is this caused with updating matched entries?) * I cannot walk across suggested entries. * Focus is staying on location bar input field if I input up/down key repeatedly.
Reporter | ||
Updated•11 years ago
|
Keywords: regression
Assignee | ||
Updated•11 years ago
|
status-firefox25:
--- → ?
Updated•11 years ago
|
tracking-firefox25:
--- → ?
Keywords: regressionwindow-wanted
Comment 1•11 years ago
|
||
This affects all form input drop downs, not just the location bar's.
Comment 2•11 years ago
|
||
Regression-hunted this, caused by bug 893973, verified by a local backout against m-c tip. Masayuki, I think we may want to back out considering how many people are affected...
Updated•11 years ago
|
Comment 3•11 years ago
|
||
> Regression-hunted this, caused by bug 893973 I assume you're referring to this patch: attachment 778350 [details] [diff] [review] ("nsIWidget::SetInputContext() should check it has focus before calling secure event input API")
Assignee | ||
Comment 4•11 years ago
|
||
(In reply to Steven Michaud from comment #3) > > Regression-hunted this, caused by bug 893973 > > I assume you're referring to this patch: attachment 778350 [details] [diff] [review] > [review] ("nsIWidget::SetInputContext() should check it has focus before > calling secure event input API") No, I believe it's https://hg.mozilla.org/mozilla-central/rev/197b12188f3a. Steven, do you mind logging in to IRC if possible?
Comment 5•11 years ago
|
||
(In reply to :Ehsan Akhgari (needinfo? me!) from comment #4) > (In reply to Steven Michaud from comment #3) > > > Regression-hunted this, caused by bug 893973 > > > > I assume you're referring to this patch: attachment 778350 [details] [diff] [review] > > [review] ("nsIWidget::SetInputContext() should check it has focus before > > calling secure event input API") > > No, I believe it's https://hg.mozilla.org/mozilla-central/rev/197b12188f3a. > Steven, do you mind logging in to IRC if possible? Correct. Backing that out fixes it.
Comment 6•11 years ago
|
||
Sigh, go ahead. I can't imagine why that patch causes trouble, but I'll figure that out later.
Assignee | ||
Comment 7•11 years ago
|
||
I backed out the offending patch: https://hg.mozilla.org/mozilla-central/rev/fd03bb4d1e48
Assignee: nobody → ehsan
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Flags: needinfo?(masayuki)
Updated•11 years ago
|
status-firefox24:
--- → unaffected
Updated•11 years ago
|
Comment 9•11 years ago
|
||
Reproduced in nightly 2013-07-31. Verified fixed FF 25b3 Mac OS X 10.8.4
You need to log in
before you can comment on or make changes to this bug.
Description
•