Avoid the url prediction query in UnifiedComplete

RESOLVED FIXED in Firefox 42

Status

()

Toolkit
Places
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mak, Assigned: mak)

Tracking

Trunk
mozilla42
Points:
---
Dependency tree / graph
Bug Flags:
firefox-backlog +

Firefox Tracking Flags

(firefox42 fixed)

Details

(Whiteboard: [unifiedcomplete][fxsearch])

Attachments

(1 attachment)

We can simplify the code and make it more maintainable if we remove the prediction and just run a faster query.
The query only runs when there's a slash in the search text so it's not a critical path.
Flags: firefox-backlog+
Created attachment 8629579 [details] [diff] [review]
patch v1

This simplifies some code indeed.
Attachment #8629579 - Flags: review?(adw)
Priority: P2 → P3

Updated

2 years ago
Attachment #8629579 - Flags: review?(adw) → review+

Comment 2

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/dc7c99a616b9
https://hg.mozilla.org/mozilla-central/rev/dc7c99a616b9
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox42: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.