Closed Bug 1521629 Opened 3 years ago Closed 3 years ago

Port search() to UrlbarInput.jsm


(Firefox :: Address Bar, enhancement, P1)




Firefox 67
Tracking Status
firefox67 --- fixed


(Reporter: rrosario, Assigned: rrosario)




(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
Port search() to UrlbarInput.jsm r=dao
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.