Closed Bug 1945999 Opened 1 year ago Closed 1 year ago

Make Suggest initialization happen in one place

Categories

(Firefox :: Address Bar, task, P3)

task

Tracking

()

RESOLVED FIXED
137 Branch
Tracking Status
firefox137 --- fixed

People

(Reporter: adw, Assigned: adw)

References

Details

(Whiteboard: [sng])

Attachments

(1 file)

This is the first step toward fixing bug 1937250.

Right now, Suggest initialization is split over two different functions:

  1. UrlbarPrefs.updateFirefoxSuggestScenario() initializes the default Suggest "scenario", meaning the default values of a handful of prefs.
  2. QuickSuggest.init() initializes all the various sub-features of Suggest.

Both of these functions are called by BrowserGlue, first 1 and then 2.

We can move 1 into 2 so that BrowserGlue only needs to call QuickSuggest.init().

There are no other callers of updateFirefoxSuggestScenario() other than tests.

Blocks: 1946894
Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/015d5fd1c79f Make Suggest initialization happen in one place. r=daisuke
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 137 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: