Closed Bug 1715484 Opened 3 years ago Closed 3 years ago

Stop syncing browser.urlbar.resultBuckets

Categories

(Firefox :: Address Bar, defect, P1)

defect
Points:
2

Tracking

()

RESOLVED FIXED
91 Branch
Iteration:
91.1 - May 31 - Jun 13
Tracking Status
firefox91 --- fixed

People

(Reporter: mak, Assigned: bugzilla)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

We may end up syncing an old definition onto a new one, maybe the new value contains just added buckets, like inputhistory and when we sync up an old version over it we lose the new bucket. It looks like a footgun and maybe it's already causing bugs.

We should stop syncing, and likely also change the pref name to avoid old versions syncing over it.

Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Iteration: --- → 91.1 - May 31 - Jun 13
Points: --- → 2
Priority: P2 → P1

I renamed the pref to resultBucketSet. This is similar enough that we don't have to change a lot of code. Functions and comments that refer to resultBuckets now clearly refer to resultBucketSet. As Marco pointed out in the bug, the renaming is necessary to avoid conflicts with old versions.

Blocks: 1715822
Pushed by htwyford@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c98845429ae7
Stop syncing browser.urlbar.resultBuckets and rename the pref. r=adw
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: