384761 - Camino doesn't set default homepage pref until the homepage is loaded

Status: RESOLVED FIXED

(Camino Graveyard :: Preferences, defect)

Description

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Once upon a time (or at least I think so), we would write the homepage pref (browser.startup.homepage) with the default homepage URL (HomePageURL from WebsiteDefaults.strings) upon launch, or something like that, such that if you opened the prefs, the default homepage URL was filled in in the box.

Now, it seems we don't write that URL to the prefs until first quit, so if someone opens the prefs (to change something else entirely) and doesn't set a page, they end up with no homepage set.

I'd much rather (and I'm thinking Sam would agree here, for stats) make sure the default homepage gets set correctly in the beginning, and anyone who wants to have a blank homepage can simply delete what we've prefilled, which is a minor inconvenience.

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

More clear STR:

1. Launch Camino with a fresh profile
2. Open prefs (if you're testing a build that remembers last-selected prefPane, you want General to have been last-selected)
3. Make no changes, close prefs
4. Quit Camino
5. Relaunch Camino
6. Open prefs

In affected builds, the homepage is not prefilled when/before opening the General prefs for the first time and then is saved as blank.

After Ian called me crazy, I did some more extensive checking.  This is only broken on the branches, and only on either 10.3 or PPC.  

Either that, or something is totally crazy about my 10.3 Mac, but I have no other 10.3 or PPC Macs to check, and I can reproduce this reliably.

Comment 2

[froodian (Ian Leue)]

I saw this on my 10.4 PPC Mac, just before handing it off to my girlfriend.  So looks like it's PPC, not 10.3.

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

I just saw this testing 1.5.3 ML on 10.4/Intel.  I am seriously losing my mind.

Comment 4

[Stuart Morgan]

Just so no-one reaches the point where they voluntarily commit themselves, the following steps will trigger the empty-home-page behavior:
1) Launch Camino with a clean profile *and in such a way that there is an override URL that prevents the homepage from opening*. I suspect the latter is the reason for both the suspicion of insanity and the appearance of this being branch only (the welcome page being the obvious way to get that behavior, which happens only with release builds)
2) Open prefs, without having ever opened a tab or window that would trigger a homepage load.

Taking; patch shortly.

Comment 5

[Stuart Morgan]

Attachment: fix

Comment 6

[froodian (Ian Leue)]

Comment on attachment 288504 [details] [diff] [review]
fix

aha! r=me

Comment 7

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 288504 [details] [diff] [review]
fix

sr=pink

I saw this setting up my wiped leopard machine.

Comment 8

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Is this something we want to take on the 1_5 branch for new users (once it has baked, etc)?

Comment 9

[Stuart Morgan]

Landed on trunk and MOZILLA_1_8_BRANCH.

Taking for 1.5.x sounds fine to me, since it's a very low-risk fix.

Comment 10

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Stuart, this doesn't apply cleanly (hunk 1) on the 1_5 branch, and there are enough changes in context that I'm not sure where that hunk is supposed to go.  Can you spin a new patch?

Comment 11

[Stuart Morgan]

Attachment: 1.5.5 version

Applied to the 1_5 branch. My 1.5.x tree isn't set up to build, so this hasn't actually be tested.

Comment 12

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Checked in on the CAMINO_1_5_BRANCH.  I built/tested on 10.5/Intel and 10.3.9/PPC.