Closed
Bug 652016
Opened 14 years ago
Closed 14 years ago
When switching channels, I get a failed update
Categories
(Toolkit :: Application Update, defect)
Toolkit
Application Update
Tracking
()
VERIFIED
FIXED
mozilla5
People
(Reporter: christian, Assigned: robert.strong.bugs)
References
Details
(Whiteboard: [channel-switcher])
Attachments
(7 files, 3 obsolete files)
216.40 KB,
image/png
|
Details | |
213.56 KB,
image/png
|
Details | |
209.51 KB,
image/png
|
Details | |
219.38 KB,
image/png
|
Details | |
12.39 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
3.93 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
12.40 KB,
patch
|
robert.strong.bugs
:
review+
asa
:
approval-mozilla-aurora+
asa
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
1. Check the about dialog on an older build on the aurora channel 2. Make sure there is an update already downloaded and ready to install (the button says "Apply Update") 3. Change to the "Beta" channel 4. Click "Apply and Update" 5. Notice an update downloads, even though one was 100% downloaded previously 6. Get an "Update failed" message
![]() |
Assignee | |
Comment 4•14 years ago
|
||
limi, what UX behavior should we have? For example, if there is an update ready to apply should the user have the ability to change the channel?
Update log attached. I haven't tried to see if it works itself out yet.
![]() |
Assignee | |
Comment 6•14 years ago
|
||
No more files are needed... btw: the update.log is from the previously applied update and there is no log file for the failed download.
Attachment #527693 -
Attachment is obsolete: true
Comment 7•14 years ago
|
||
(In reply to comment #4) > limi, what UX behavior should we have? For example, if there is an update ready > to apply should the user have the ability to change the channel? Yes, you should always be able to change the channel. If there's an update queued for e.g. the Beta channel and you want to switch to the Aurora channel, we should discard the update and get the Aurora one instead.
![]() |
Assignee | |
Updated•14 years ago
|
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
![]() |
Assignee | |
Comment 8•14 years ago
|
||
![]() |
Assignee | |
Updated•14 years ago
|
Whiteboard: [channel-switcher]
Updated•14 years ago
|
Comment 10•14 years ago
|
||
will look again at a patch approval request but not going to track this through this flag.
![]() |
Assignee | |
Comment 12•14 years ago
|
||
I've got a patch but I've had trouble finishing up the tests and my eye problems flared up again... still hopeful to have it done today.
![]() |
Assignee | |
Comment 13•14 years ago
|
||
Dave, I'll followup with tests later after I get replacement eyeglasses
Attachment #528565 -
Attachment is obsolete: true
Attachment #532716 -
Flags: review?(dtownsend)
![]() |
Assignee | |
Comment 14•14 years ago
|
||
Attachment #532716 -
Attachment is obsolete: true
Attachment #532716 -
Flags: review?(dtownsend)
Attachment #532863 -
Flags: review?(dtownsend)
Comment 16•14 years ago
|
||
Comment on attachment 532863 [details] [diff] [review] main patch rev1 Review of attachment 532863 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/mozapps/update/nsUpdateService.js @@ +500,5 @@ > var channelChangeFile = dir.clone(); > channelChangeFile.append(FILE_CHANNELCHANGE); > + if (!channelChangeFile.exists()) > + channelChangeFile.create(Ci.nsILocalFile.NORMAL_FILE_TYPE, > + FileUtils.PERMS_FILE); nit: braces around this
Attachment #532863 -
Flags: review?(dtownsend) → review+
Comment 17•14 years ago
|
||
Comment on attachment 532895 [details] [diff] [review] tests rev1 Review of attachment 532895 [details] [diff] [review]: -----------------------------------------------------------------
Attachment #532895 -
Flags: review?(dtownsend) → review+
![]() |
Assignee | |
Comment 18•14 years ago
|
||
Comment on attachment 532863 [details] [diff] [review] main patch rev1 Requesting approval for Aurora and Beta before landing on m-c since the code is only exercised on Aurora and Beta
Attachment #532863 -
Flags: approval-mozilla-beta?
Attachment #532863 -
Flags: approval-mozilla-aurora?
Comment 19•14 years ago
|
||
Missed this approval for aurora before the merge, but if you land it on central now, it'll get picked up in next week's aurora merge for FF6 and the world will be a happier place for Aurora users at least. We can take it on beta after that, or wait 6 weeks for it to get there on its own, based on the risk/reward calculus for those users.
![]() |
Assignee | |
Comment 21•14 years ago
|
||
Pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/195b5dc114c5 http://hg.mozilla.org/mozilla-central/rev/062dccbd5674
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Flags: in-litmus-
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
![]() |
Assignee | |
Updated•14 years ago
|
Attachment #532863 -
Flags: approval-mozilla-beta?
Attachment #532863 -
Flags: approval-mozilla-aurora?
![]() |
Assignee | |
Comment 22•14 years ago
|
||
Comment on attachment 533340 [details] [diff] [review] main patch rev2 - updated to comments Readding the beta approval request
Attachment #533340 -
Flags: approval-mozilla-beta?
![]() |
Assignee | |
Updated•14 years ago
|
Attachment #533340 -
Flags: approval-mozilla-aurora?
Updated•14 years ago
|
Attachment #533340 -
Flags: approval-mozilla-beta?
Attachment #533340 -
Flags: approval-mozilla-beta+
Attachment #533340 -
Flags: approval-mozilla-aurora?
Comment 23•14 years ago
|
||
Comment on attachment 533340 [details] [diff] [review] main patch rev2 - updated to comments Please land this change on both Aurora and Beta. (In the future, getting changes in during Aurora will save you this extra step.)
Attachment #533340 -
Flags: approval-mozilla-aurora+
![]() |
Assignee | |
Comment 24•14 years ago
|
||
Pushed to mozlla-aurora http://hg.mozilla.org/releases/mozilla-aurora/rev/21f2117c6eb1 http://hg.mozilla.org/releases/mozilla-aurora/rev/90a0d1cdff11 Pushed to mozlla-beta http://hg.mozilla.org/releases/mozilla-beta/rev/a9152271fee4 http://hg.mozilla.org/releases/mozilla-beta/rev/546612551fbd
Target Milestone: mozilla6 → mozilla5
Comment 25•14 years ago
|
||
Mozilla/5.0 (Windows NT 5.1; rv:5.0) Gecko/20100101 Firefox/5.0 - Firefox 5 Beta - build 3 Verified the issue on WinXP and it's no longer reproducible. The update testing was done between Aurora 6.0a2 to Beta 5.0b3.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•