After uninstalling a default add-on panel the 4th panel is the new default

VERIFIED FIXED in Firefox 42

Status

()

defect
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: sebastian, Assigned: sebastian)

Tracking

unspecified
Firefox 43
All
Android
Points:
---

Firefox Tracking Flags

(firefox42 verified, firefox43 verified, fennec42+)

Details

Attachments

(1 attachment)

STR:
* Install an add-on that adds a panel to the home screen
* Make this panel the default panel
* Uninstall the add-on

Expected result:
The first panel should be the default panel.

Actual result:
The 4th panel is the new default panel.


We recently changed the default ordering of the home panels and with that made the first panel the new default panel on phones and tablets. There's probably some code running on uninstall that tries to restore the default of the old home panel order.

Updated

4 years ago
tracking-fennec: --- → ?
Assignee: nobody → s.kaspari
tracking-fennec: ? → 42+
Status: NEW → ASSIGNED
Bug 1196198 - Use ordered list to find new default panel. r?mhaigh
Attachment #8654821 - Flags: review?(mhaigh)
Comment on attachment 8654821 [details]
MozReview Request: Bug 1196198 - Use ordered list to find new default panel. r?mhaigh

https://reviewboard.mozilla.org/r/17759/#review15981
Attachment #8654821 - Flags: review?(mhaigh) → review+
Comment on attachment 8654821 [details]
MozReview Request: Bug 1196198 - Use ordered list to find new default panel. r?mhaigh

Approval Request Comment
[Feature/regressing bug #]: We updated the default order of panels in Fx42.
[User impact if declined]: After uninstalling an add-on that added a default panel the new default panel is kind of random (the 4th most of the time).
[Describe test coverage new/current, TreeHerder]: Manual testing.
[Risks and why]: Low risk. Instead of finding a new default panel in an unsorted list, we are now searching a new default panel in the sorted list of panels.
[String/UUID change made/needed]: -
Attachment #8654821 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/9af574254ea0
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Comment on attachment 8654821 [details]
MozReview Request: Bug 1196198 - Use ordered list to find new default panel. r?mhaigh

ok, let's do it!
Attachment #8654821 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Install wikipedia add-on, make it default and then uninstall it, "Top Sites" will appear as the default one
Verified as fixed using:
Device: Moto X (Android 4.4)
Build: Firefox for Android  43.0a1 (2015-09-06) and Firefox for Android 42.0a2 (2015-09-07)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.