Closed
Bug 1303624
Opened 8 years ago
Closed 8 years ago
Remove typeAheadResult support from autocomplete
Categories
(Toolkit :: Autocomplete, defect, P3)
Toolkit
Autocomplete
Tracking
()
RESOLVED
FIXED
mozilla52
People
(Reporter: mak, Assigned: mak)
Details
(Whiteboard: [fxsearch])
Attachments
(1 file)
We were using this for old autocomplete, now we don't use it anymore. Since the awesomebar was the only consumer and the reason it was added, it's worth to remove this special case to be able to simplify autocomplete code a little bit.
Assignee | ||
Updated•8 years ago
|
Summary: Remote typeAheadResult support → Remove typeAheadResult support
Assignee | ||
Updated•8 years ago
|
Summary: Remove typeAheadResult support → Remove typeAheadResult support from autocomplete
Comment hidden (mozreview-request) |
Comment 2•8 years ago
|
||
mozreview-review |
Comment on attachment 8793368 [details] Bug 1303624 - Remove typeAheadResult support from autocomplete. https://reviewboard.mozilla.org/r/80110/#review78886 Nice
Attachment #8793368 -
Flags: review?(adw) → review+
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/4fb71c5c1e6b Remove typeAheadResult support from autocomplete. r=adw
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4fb71c5c1e6b
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Comment 5•8 years ago
|
||
Mark 51 as fix-optional. If it's worth uplifting to 51, feel free to nominate it.
Assignee | ||
Comment 6•8 years ago
|
||
no, this is just code cleanup.
You need to log in
before you can comment on or make changes to this bug.
Description
•