Closed
Bug 873429
Opened 12 years ago
Closed 12 years ago
Form assistant suggestions is not dismissed while entering different letters
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox21 unaffected, firefox22 unaffected, firefox23+ verified, firefox24+ verified)
VERIFIED
FIXED
Firefox 24
Tracking | Status | |
---|---|---|
firefox21 | --- | unaffected |
firefox22 | --- | unaffected |
firefox23 | + | verified |
firefox24 | + | verified |
People
(Reporter: ioana.chiorean, Assigned: Margaret)
References
Details
(Keywords: regression, reproducible)
Attachments
(1 file)
(deleted),
patch
|
mfinkle
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Build: Aurora 23.0a2 05/16
Device: Asus Eee Transformers
OS: Android 4.0.3
Steps to Reproduce:
1. Go to gmail.com
2. Enter a user name - tap enter - repeat with a different user name
3. Enter a 3rd user name - starting with different letter from the above 2
Expected Results:
- no suggestion present
Actual Results:
- the first 2 names as present as suggestions
Please see video - http://www.youtube.com/watch?v=pq2Wnc4mXVM . Suggestions can be dismissed if you will tap outside the form.
Updated•12 years ago
|
tracking-fennec: --- → ?
Comment 1•12 years ago
|
||
The regression window for this issue is:
1. mozilla central
good build: 11.05.2013
bad build: 12.05.2013
pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=179e29a23c56&tochange=d68224f5325b
2. inbound
good build: 1368163665
bad build: 1368166544
pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=6035591d40f8&tochange=96fe69d53f35
Keywords: regressionwindow-wanted
Comment 2•12 years ago
|
||
Looks like async form history broke this.
Updated•12 years ago
|
tracking-firefox23:
--- → ?
tracking-firefox-esr17:
? → ---
Assignee | ||
Comment 4•12 years ago
|
||
Yo dawg, there are a lot of callbacks in this code. Turns out there were two slight logic errors that caused us to never call _hideFormAssistPopup when we should.
Attachment #751868 -
Flags: review?(mark.finkle)
Comment 5•12 years ago
|
||
Comment on attachment 751868 [details] [diff] [review]
patch
Off topic: I just realized I have a strong dislike for "fat arrow" function syntax. It's the work of the devil.
Attachment #751868 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 751868 [details] [diff] [review]
patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 566746
User impact if declined: form autocomplete popup doesn't hide when it should
Testing completed (on m-c, etc.): tested locally (just landed on inbound)
Risk to taking this patch (and alternatives if risky): low-risk, isolated small logic fixes
String or IDL/UUID changes made by this patch: n/a
Attachment #751868 -
Flags: approval-mozilla-aurora?
Comment 8•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
Updated•12 years ago
|
Updated•12 years ago
|
Attachment #751868 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 9•12 years ago
|
||
Updated•12 years ago
|
Updated•11 years ago
|
tracking-fennec: ? → ---
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•