Closed Bug 1196198 Opened 5 years ago Closed 5 years ago

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

Categories

(Firefox for Android :: Awesomescreen, defect)

All
Android
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 43
Tracking Status
firefox42 --- verified
firefox43 --- verified
fennec 42+ ---

People

(Reporter: sebastian, Assigned: sebastian)

Details

Attachments

(1 file)

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.
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: 5 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.