Closed Bug 1628015 Opened 2 years ago Closed 1 year ago

Remove references to PlacesSearchAutocompleteProvider from UnifiedComplete

Categories

(Firefox :: Address Bar, task, P3)

task
Points:
2

Tracking

()

RESOLVED FIXED

People

(Reporter: harry, Unassigned)

References

Details

(Whiteboard: [fixed by bug 1640045])

Attachments

(1 obsolete file)

Suggested by Marco here.

We should remove references to PlacesSearchAutocompleteProvider from UnifiedComplete so we can refactor PlacesSearchAutocompleteProvider. We might want to just fold it into UrlbarProviderSuggestions or we could rename it UrlbarSearchUtils and move it to browser/components/urlbar.

Priority: -- → P3
Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Iteration: --- → 79.1 - June 1 - June 14
Iteration: 79.1 - June 1 - June 14 → 79.2 - June 15 - June 28

Hah, I assigned my Phabricator patch the wrong bug number! I haven't yet produced a patch for this. I'll make the necessary change in Phabricator.

Comment on attachment 9155738 [details]
Bug 1628015 - Resolve incompatibility between Search Tips and Paste & Go. r?mak

Revision D79174 was moved to bug 1643687. Setting attachment 9155738 [details] to obsolete.

Attachment #9155738 - Attachment is obsolete: true

I think you're clashing with Drew's https://phabricator.services.mozilla.com/D79244

Assignee: htwyford → nobody
Status: ASSIGNED → NEW
Iteration: 79.2 - June 15 - June 28 → ---

Bug 1640045 removed PlacesSearchAutocompleteProvider.

Status: NEW → RESOLVED
Closed: 1 year ago
Depends on: 1640045
Resolution: --- → FIXED
Whiteboard: [fixed by bug 1640045]
You need to log in before you can comment on or make changes to this bug.