Closed Bug 1303624 Opened 8 years ago Closed 8 years ago

Remove typeAheadResult support from autocomplete

Categories

(Toolkit :: Autocomplete, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox51 --- wontfix
firefox52 --- fixed

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.
Summary: Remote typeAheadResult support → Remove typeAheadResult support
Summary: Remove typeAheadResult support → Remove typeAheadResult support from autocomplete
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
https://hg.mozilla.org/mozilla-central/rev/4fb71c5c1e6b
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Mark 51 as fix-optional. If it's worth uplifting to 51, feel free to nominate it.
no, this is just code cleanup.
You need to log in before you can comment on or make changes to this bug.