Custom Page has a double entry for Start Page option

VERIFIED FIXED in Firefox 9

Status

Fennec Graveyard
General
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: xti, Assigned: mbrubeck)

Tracking

({regression})

Firefox 9
Firefox 9
ARM
Android
regression

Details

(Whiteboard: [pushed])

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 563091 [details]
screenshot

Build ID: Mozilla/5.0 (Android;Linux armv7l;rv:9.0a2)Gecko/20110928
Firefox/9.0a2 Fennec/9.0a2
Device: Samsung Galaxy S
OS: Android 2.2

Build ID: Mozilla/5.0 (Android;Linux armv7l;rv:10.0a1)Gecko/20110928
Firefox/10.0a1 Fennec/10.0a1
Device: Samsung Galaxy S
OS: Android 2.2

Steps to reproduce:
1. Open Fennec App
2. Browse to www.google.com
3. Go to Preferences > Start Page 
4. Select Custom Page
5. Open a new tab and go to www.mozilla.com
6. Repeat step 3

Expected result:
After step 4, no Start Page entry is duplicated.

Actual result:
Custom Page option is listed twice as you can see on the attached screenshot.
Assignee: nobody → wjohnston
(Assignee)

Comment 1

6 years ago
Created attachment 563166 [details] [diff] [review]
patch

Bug 658278 removed the prefs-languages element, but the preferences initialization code was still looking for it, so it ended up initializing stuff more than once.
Assignee: wjohnston → mbrubeck
Status: NEW → ASSIGNED
Attachment #563166 - Flags: review?(wjohnston)
(Assignee)

Comment 2

6 years ago
Nominating for tracking-fennec.  This regression affects Firefox 9 and may cause other subtler bugs.  We should request approval to land this in Aurora when it has been reviewed and pushed.
Blocks: 658278
tracking-fennec: --- → ?
status-firefox9: --- → affected
Keywords: regression
Version: Trunk → Firefox 9
tracking-fennec: ? → 9+
Comment on attachment 563166 [details] [diff] [review]
patch

Thanks matt!
Attachment #563166 - Flags: review?(wjohnston) → review+
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/7cc26b83a57c
Whiteboard: [pushed]
(Assignee)

Comment 5

6 years ago
Comment on attachment 563166 [details] [diff] [review]
patch

Requesting approval for Aurora 9.  This fixes a regression introduced in Firefox 9.  The symptoms are not serious, but they are user-visible, and the nature of the regression makes me think there may be more subtle symptoms that we haven't found yet.  The fix is mobile-only, very localized, and safe.  It removes a attempt to reference a non-existent XUL element.
Attachment #563166 - Flags: approval-mozilla-aurora?

Comment 6

6 years ago
https://hg.mozilla.org/mozilla-central/rev/7cc26b83a57c
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Target Milestone: --- → Firefox 10

Updated

6 years ago
Attachment #563166 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 7

6 years ago
Pushed to Aurora for Firefox 9:
https://hg.mozilla.org/releases/mozilla-aurora/rev/b9a6752584aa
status-firefox9: affected → fixed
Target Milestone: Firefox 10 → Firefox 9
(Reporter)

Comment 8

6 years ago
Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a2)Gecko/20110930
Firefox/9.0a2 Fennec/9.0a2
Device: Samsung Galaxy S
OS: Android 2.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.