Closed Bug 1521629 Opened 3 years ago Closed 3 years ago

Port search() to UrlbarInput.jsm

Categories

(Firefox :: Address Bar, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: rrosario, Assigned: rrosario)

References

Details

Attachments

(1 file)

Bug 1480503 added the search() function (for search shortcuts) to urlbarBindings.xml but that's going away. That code needs to be ported to UrlbarInput.jsm.

Blocks: 1480503
No longer blocks: 1514352

Would be nice to unify this with typeRestrictToken. The differences seem somewhat arbitrary.

Iteration: --- → 67.1 - Jan 28 - Feb 10
Priority: -- → P2
Assignee: nobody → rrosario
Priority: P2 → P1

I thought this would be quick to do but, after looking at it, I realize it depends on things in in UrlbarInput.jsm yet.

Assignee: rrosario → nobody
Iteration: 67.1 - Jan 28 - Feb 10 → ---
Component: Activity Streams: Newtab → Address Bar

What things specifically?

Flags: needinfo?(rrosario)

(In reply to Dão Gottwald [::dao] from comment #3)

What things specifically?

The first things I ran into were this.whichSearchSuggestionsNotification and this.popup. I'm not very familiar with everything going on there but I'm willing to help out with some guidance.

Flags: needinfo?(rrosario)

The search suggestion notification isn't implemented yet, so that whole block can be disregarded for now. It doesn't seem crucial to the main purpose of this method.

In that case, I'll have a patch shortly :)

Assignee: nobody → rrosario

MozReview-Commit-ID: 70RV8IdoAKc

Pushed by rrosario@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5b4294b58904
Port search() to UrlbarInput.jsm r=dao
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
You need to log in before you can comment on or make changes to this bug.