Closed Bug 102313 Opened 19 years ago Closed 16 years ago

milestone home page override is very annoying - browser.startup_homepage_overide

Categories

(SeaMonkey :: Installer, enhancement)

enhancement
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.7beta

People

(Reporter: bbaetz, Assigned: danm.moz)

References

Details

Attachments

(1 file)

This start page setting is getting on my nerves.

I routinely switch between 0.9.4 and the trunk. When I do this, I get the
"welcome to mozilla" page. Every single time.

There is no way to turn this off without editing the code. Please, either give
me a pref to turn this off always, or teach the version comparison code to not
just do a plain string compare, so that I can set the pref to version 99.99.99.
> There is no way to turn this off without editing the code.

Surely putting:
user_pref("startup.homepage_override_url", "about:blank");
in users.js would do the trick?

Gerv
Nope. Anything other than the current mstone value sends it to
mozilla.org/start.html

It's annoying as all heck for those of us already "in the know".
sounds like a dup of bug 99899
No, it's not :-) Bradley want's to be able to turn of the start_page_override_url.

Would it fix it to get the checking code to only show the URL if the version
number had increased? And not to reset the version number in the pref if it was
a decrease? This way your version number pref would stay at the highest value
that had ever been run, and none of the lower-valued ones would show the start_page.

Gerv
That WOULD help because then we could set it to 0.9.9+ and never see it till 1.0

Make it so, Mr. Markham. ;)
I know nothing about that bit of code at all :-(

Gerv
Well quit making pointless entries into bugzilla and LEARN .;)
gerv: well, I could set it to my home page, and then I wouldn't notice the
difference. However, I've used this profile with 6.1 (yes, yes, I know), and so
I have an entry:
user_pref("startup.homepage_override_url","http://home.netscape.com/bookmark/6_1/startuppage.html");
But when I start blizzard's 0.9.4 rpms, I get the mozilla start page. Something
else is going on, and so that trick won't work.
(That page resizes. Do you know how annoying that was before I discovered it
last night?)

The problem with the version number is that theres no real definition of
increasing. If you ever run a netscape build, it would be set to 6.x, and then
would never show the mozilla start page. While I know that we don't support
that, I suspect that first-time mozilla people would use an existing profile. Or
do the commercial releases use the mozilla milestone value?
jag: any comment here?

Gerv
Samir: I'll take this bug (but feel free to take it back if you really want it).
Assignee: sgehani → jaggernaut
Set the following pref to any string in your user.js and then run the browser
twice.  The second time onwards you should never see the homepage override
mechanism kick in no matter whether you use mozilla or ns builds.  

Pref you want is: browser.startup.homepage_override.mstone

No, that makes the start page _always_ show!

If I do that, then we see that the pref milestone doesn't match the current one,
and display the page, but because its now in user.js, it never gets updated.
I'm pretty sure Samir meant to say prefs.js.
But that doesn't help either - it will still always show when swapping between
0.9.4/trunk. If I set the url to "foo", then foo != "rv:0.9.4", so it displays.
And then the code writes out the setting of rv:0.9.4, and we're back where we
started.
Yep. So, here's my proposal: add another pref that allows developers to turn
this feature off, but it'll be set by default to always display the url when the
mstone changes.
Rather than ANOTHER pref, why not just check to see if either the override URL
or mstone value is equal to a predefined value, say "none", and if so, don't
override. If != "none" (or whatever) then continue with checking if the mstone
has changed, etc... ?
"latest". I like your idea.
Glad to hear it. :) I'll take a look at the code tomorrow to see if I can make a 
simple patch for it.
Ok, I guess the relevant file and position is currently
http://lxr.mozilla.org/seamonkey/source/xpfe/browser/src/nsBrowserInstance.cpp#891
which is where the milestone check block starts. SO, maybe we have this section
(I'm including line numbers so it's easier to find when viewing the source):

905  // failed to get pref -or- saved milestone older than current milestone, 
906  // write out known current milestone and show URL this time
907  if (NS_FAILED(rv) || 
908      !(currMilestone.Equals(savedMilestone)))
909  {
910    // update milestone in "homepage override" pref
911    aPrefService->SetCharPref(PREF_HOMEPAGE_OVERRIDE_MSTONE, 
912                              currMilestone.get());
913    return PR_TRUE;
914  }
915
916  // don't override if saved and current are same
917  return PR_FALSE;


So I guess lines 907 and 908 hold they key.
Actually, I forgot to mention you would need to set general.useragent.misc
 to the same string and I did mean user.js.
Yeah, I realized you did mean user.js after thinking about it some more. The
missing factor was indeed overriding the misc string in the same way.
So, I looked at this. We initialise http at startup, for the sole purpose of
getting the a single pref. Yuck.

Anyway, I don't want to change my browser's useragent to get rid of this.

Would people object to a HOMEPAGE_OVERRIDE_MILESTONE of 'latest' meaning that we
don't show this page? IF so, I'll code it up, and try to get it into 0.9.5.
If we are initialising HTTP at startup solely because of this feature, then it
needs a total rethink.

Gerv
according to darin its loaded at startup for a couple of other reasons.

Thats no excuse for just not reading the pref directly though, but thats a
separate issue to this bug, which I want fixed for 0.9.5.
We could also concatenate all the versions we've seen into the pref and then
scan for it. That way it would do the right thing even for folks who don't know
about this magic value, although at the cost of a pretty long pref eventually.
QA Contact: bugzilla → ktrina
-> Future
Target Milestone: --- → Future
*** Bug 122940 has been marked as a duplicate of this bug. ***
*** Bug 123810 has been marked as a duplicate of this bug. ***
Bug 99441 covers the problem where a "Welcome to Netscape 6.1!" page appears
instead of Mozilla's new-milestone page.
might be interesting:
i use server based profiles (w95b, w98se, Mozilla v 0.9.6 - Build 2002041711)
and  always got the start page on a client where i had installed windows on
drive 'd:' instead of 'c:' (and the folder 'program files' that contained the
mozilla files). now mozilla is installed on all clients on the same drive and in
the same folder and the welcome-page doesnt appear any longer.
with netscape 4.x i had similar problems. the 'drive d: netscape' always forgot
the cache and the cookie settings.
why not change the homepage override ?

currently :
Build V1 -> Profile V2 -> V2 changed to V1 in the profile -> Mozilla/NS Start Page

should be:
Build V1 -> Profile V2 -> no start page override and V2 remeains in the profile
Build V2 -> Profile V1 -> Mozilla/NS start page 
*** Bug 235981 has been marked as a duplicate of this bug. ***
OS->All.

Tweaking summary so I could have found it before filing my duplicate.

As much as I hate this behaviour myself, I don't believe that this bug's
severity is appropriate.  I'd marked mine as an enhancement calling for a hidden
preference because I see what the browser does as a good thing for people who
aren't aware of how they can get involved.  Only people who know enough about
the product to be able to track it down should (implicitly acknowledging that
they're already involved) should be able to turn it off.  Further, even if it's
a legitimate bug (I'm not sure how it can be) it's certainly not, objectively
speaking, "major" by any stretch of the imagination that I can see...

OS: Linux → All
Summary: milestone home page override is very annoying → milestone home page override is very annoying - browser.startup_homepage_overide
Oh for petesake. This isn't difficult, and it is annoying.
Attachment #142634 - Flags: superreview?(jag)
Attachment #142634 - Flags: review?(bugs)
I'd give this patch r+, but it wouldn't mean anything. He's got an excellent
point, however.
Comment on attachment 142634 [details] [diff] [review]
magic value "ignore" skips the milestone homepage hijack

r+sr=jag
Attachment #142634 - Flags: superreview?(jag)
Attachment #142634 - Flags: superreview+
Attachment #142634 - Flags: review?(bugs)
Attachment #142634 - Flags: review+
Assignee: jag → danm-moz
Severity: major → enhancement
Hardware: PC → All
Target Milestone: Future → mozilla1.7beta
Thanks, jag. Fixed in 1.7b: set your browser.startup.homepage_override.mstone
pref to "ignore" (by hand) and you get to keep your homepage.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
QA Contact: ktrina → benc
Product: Browser → Seamonkey
Duplicate of this bug: 357020
You need to log in before you can comment on or make changes to this bug.