Closed
Bug 477908
Opened 16 years ago
Closed 16 years ago
Set a smaller value for DOWNLOAD_BACKGROUND_INTERVAL for non official builds
Categories
(Toolkit :: Application Update, defect)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
mozilla1.9.2a1
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
Details
Attachments
(1 file)
(deleted),
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
When downloading an complete update for a nightly build the interval is way too long and will take hours (around 5?) to complete and often it will be the next day before the download has completed. By changing DOWNLOAD_BACKGROUND_INTERVAL from 600 to 60 for non official builds (e.g. nightly builds) the approximate time to complete should be under an hour.
Assignee | ||
Comment 1•16 years ago
|
||
Probably best to key on either the channel or official branding.
Assignee | ||
Comment 2•16 years ago
|
||
or MOZILLA_OFFICIAL
Comment 3•16 years ago
|
||
MOZILLA_OFFICIAL means "anything built on any Mozilla tinderbox" so you don't want that.
Assignee | ||
Comment 4•16 years ago
|
||
Yea... I'm a bit slow and under the weather today
Comment 5•16 years ago
|
||
If we're going down this road (and it's a good road to tread), then it'd be worth considering shortening app.update.interval too. The default check of once a day means you end up with more complete updates than is convenient for frequent nightly use. We'd have to check if that'd impact on metrics though.
Assignee | ||
Comment 6•16 years ago
|
||
I shortened the interval for nightly builds awhile ago to 8 hours
http://mxr.mozilla.org/mozilla-central/source/browser/app/firefox-branding.js#5
http://mxr.mozilla.org/mozilla-central/source/browser/branding/unofficial/pref/firefox-branding.js#5
Assignee | ||
Comment 7•16 years ago
|
||
and I notified metrics when I made the change
Comment 8•16 years ago
|
||
Pardon me, I really should pay more attention. That was in Bug 324121 ftr.
Assignee | ||
Comment 9•16 years ago
|
||
No worries
Comment 10•16 years ago
|
||
This is only affecting nightly builds and not releases, right?
Assignee | ||
Comment 11•16 years ago
|
||
correct
Assignee | ||
Comment 12•16 years ago
|
||
I went with a pref since other apps may want to specify a custom value.
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #361827 -
Flags: review?(mconnor)
Updated•16 years ago
|
Attachment #361827 -
Flags: review?(mconnor) → review+
Assignee | ||
Comment 13•16 years ago
|
||
I'll land this after the mozilla-central tree is in better shape.
btw: I gave Justin a heads up about this change for nightly updates as well.
Assignee | ||
Comment 14•16 years ago
|
||
cc'ing a bunch of people in case they are interested in adding this pref for their app updates.
Comment 15•16 years ago
|
||
It seems wrong for this to depend on branding (rather than, say, channel). I think depending on branding will make alphas behave like nightlies and betas behave like releases, which might not be what we want.
Assignee | ||
Comment 16•16 years ago
|
||
Well, the pref can be set to whatever number and for whatever type of build it wants and if an app chooses not to we fallback to the number of seconds we've been using. I agree that it *might* be nice to have a new way of setting this but that can be said about several of the prefs set in the branding prefs file as well as other code that is different based on the type of build.
Assignee | ||
Comment 17•16 years ago
|
||
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/e6d08d303d53
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
You need to log in
before you can comment on or make changes to this bug.
Description
•