Closed Bug 691579 Opened 8 years ago Closed 8 years ago

Get rid of broken multiple homepage code in startPage()

Categories

(Firefox :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 10

People

(Reporter: Margaret, Assigned: Margaret)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
We don't use the browser.startup.homepage.count pref in the UI, and if someone did happen to set it manually, it wouldn't work because this code separates homepages with "\n", when consumers expect them to be separated with "|".
Attachment #564390 - Flags: review?(gavin.sharp)
Attachment #564390 - Flags: review?(gavin.sharp) → review+
https://hg.mozilla.org/mozilla-central/rev/362872311015
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 10
Assignee: nobody → margaret.leibovic
Whiteboard: [fixed-in-fx-team]
You need to log in before you can comment on or make changes to this bug.