Closed Bug 1664402 Opened 1 year ago Closed 1 year ago

Amazon.de gets migrated into amazon.com after succesive browser.topsites.useRemoteSetting on/off switches

Categories

(Firefox :: Top Sites, defect, P1)

Firefox 81
Desktop
All
defect
Points:
1

Tracking

()

VERIFIED FIXED
82 Branch
Iteration:
82.2 - Sep 7 - Sep 20
Tracking Status
firefox-esr68 --- unaffected
firefox-esr78 --- unaffected
firefox80 --- unaffected
firefox81 --- verified
firefox82 --- verified

People

(Reporter: aflorinescu, Assigned: dao)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

[Sugested Severity:]
S2

[Description:]
Switching the browser.topsites.useRemoteSetting on/off/on, it is possible to alter the default topsite amazon.de into amazon.com.

Additionally, probably after the bug 1661008 fix, amazon.com also gets pinned by default and marked as a search shortcut. (this part is not reproducible on 81 b4)

[Environment:]
Windows 10, Ubuntu 20, Mac 10.13.6
81.0b9

[Steps:]

  1. Create a new profile with:
    user_pref("services.settings.load_dump", false);
    user_pref("browser.topsites.useRemoteSetting", true);
    user_pref("browser.search.region", "DE");
  2. Start Firefox, observe topsites -> from RS top-sites collection.
  3. Close Firefox, set browser.topsites.useRemoteSetting to false.
  4. Start Firefox, observe topsites -> legacy configuration.
  5. Close Firefox, set browser.topsites.useRemoteSetting to true.
  6. Start Firefox, observe topsites -> from RS top-sites collection.
[Actual Result:]
  1. (RS)amazon.de is listed, not pinned, not search shortcut
  2. (legacy)amazon.com is listed, pinned, search shortcut.
  3. (RS)amazon.com is listed, pinned, search shortcut.
[Expected Result:]
  1. (RS)amazon.de is listed, not pinned, not search shortcut
  2. (legacy)amazon.com is listed, pinned, search shortcut.
  3. (RS)amazon.de is listed, not pinned, not search shortcut.
Assignee: nobody → dao+bmo
Severity: -- → S3
Status: NEW → ASSIGNED
Iteration: --- → 82.2 - Sep 7 - Sep 20
Points: --- → 1
Priority: -- → P1
Blocks: 1664288
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b76c123c607d
Support migrating browser.topsites.useRemoteSetting true to false and back to true. r=mikedeboer

Comment on attachment 9175150 [details]
Bug 1664402 - Support migrating browser.topsites.useRemoteSetting true to false and back to true. r=mikedeboer

Beta/Release Uplift Approval Request

  • User impact if declined: We don't plan on switching browser.topsites.useRemoteSetting on and off and back on, but it's good to have this fix in place for users doing so manually for whatever reason. This also fixes the karma issue filed as bug 1664288.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Trivial fix
  • String changes made/needed:
Attachment #9175150 - Flags: approval-mozilla-beta?
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
See Also: → 1664502

Comment on attachment 9175150 [details]
Bug 1664402 - Support migrating browser.topsites.useRemoteSetting true to false and back to true. r=mikedeboer

Approved for 81.0rc1.

Attachment #9175150 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

This bug was verified during the PI-763 test run, but in light of latest fixes and uplifts, reverified it and comment 0 STR now work as expected on both nightly 82.0a1 2020-09-16 and 81 RC1.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.