Closed
Bug 313522
Opened 19 years ago
Closed 19 years ago
Auto-update did not happen for many beta1 users (and won't for beta2)
Categories
(Toolkit :: Application Update, defect)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
mozilla1.8rc1
People
(Reporter: darin.moz, Assigned: darin.moz)
References
Details
(Keywords: fixed1.8, relnote)
Attachments
(1 file)
(deleted),
patch
|
darin.moz
:
review+
chase
:
approval1.8rc1+
|
Details | Diff | Splinter Review |
Auto-update did not happen for many beta1 users
Some beta1 instances were not auto-updated to beta2. It seems that this had something to do with extensions that would be invalidated by beta2. We need
to resolve this problem for FF1.5 final.
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•19 years ago
|
Flags: blocking1.8rc1?
Comment 1•19 years ago
|
||
Is this the bug where we didn't offer update UI if we saw that extensions were incompatible? If so, isn't that fixed?
Assignee | ||
Comment 2•19 years ago
|
||
Yes, that's the bug I'm thinking of. I don't recall it being fixed... can you point me at the bug?
Auto-update notification from b1 to b2 never occurred for me. I had to manually check for updates and then update (fine if you know there's an update, not good otherwise...). I left it for 5 days after it was announced that the update process was switched on before giving up on waiting.
Comment 4•19 years ago
|
||
Daniel, automatic update was broken in 1.5b1 but fixed just before 1.5b2 (bug 309784), so your experience is what I would have expected.
Darin, do you have any pointers on how to duplicate this extension problem?
Comment 5•19 years ago
|
||
The bug I remember hearing about (and can't find in bugzilla, so it may not exist) was that when a user had extensions that would be disabled by the update, we didn't show the user any update UI and we didn't perform the update download.
Flags: blocking1.8rc1? → blocking1.8rc1+
Assignee | ||
Comment 6•19 years ago
|
||
Asa: right, AFAIK that's this bug and it isn't fixed.
Comment 7•19 years ago
|
||
OK. I'll try to get us a clear set of steps to reproduce. This is definitely something we need to fix.
Comment 8•19 years ago
|
||
The code in this function uses the extension manager before the em service is actually retrieved! Reorder this function so that the Listener is not created until after. A big duh.
This was affecting users with incompatibilities because this function is only called when there are incompatibilities, meaning no update prompt was being shown. When I applied this to my install of 1.5B1, it began showing the update prompt again asking me to upgrade to 1.4.1.
Attachment #200629 -
Flags: review?(darin)
Comment 9•19 years ago
|
||
The reason this is so unpredictable is:
Assume an extension, e.g. Google Toolbar with maxVersion = 1.4 is installed.
If the update timer fires, Toolbar will be discovered to be incompatible and an version-info update check performed on that extension to see if there's a newer compatible version available. That code is buggy and throws an exception, halting the entire prompting process.
If the update timer for the Toolbar extension itself fires before the app update timer fires, it will update its maxVersion to 1.4.1 and by the time the app update timer fires all will be right with the world.
Assignee | ||
Comment 10•19 years ago
|
||
Comment on attachment 200629 [details] [diff] [review]
patch
r=darin
Attachment #200629 -
Flags: review?(darin) → review+
Comment 11•19 years ago
|
||
Thanks for the quick work on this - is this ready to land? Should we land on branch/trunk at same time?
Assignee | ||
Updated•19 years ago
|
Attachment #200629 -
Flags: approval1.8rc1?
Assignee | ||
Comment 12•19 years ago
|
||
fixed-on-trunk
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Attachment #200629 -
Flags: approval1.8rc1? → approval1.8rc1+
Comment 14•19 years ago
|
||
We may want to relnote this. It's fixed *now* but not in the beta versions people have. Might head off some number of duplicate bug filings.
Keywords: relnote
Summary: Auto-update did not happen for many beta1 users → Auto-update did not happen for many beta1 users (and won't for beta2)
Comment 15•19 years ago
|
||
Is downloading the latest release and installing it over Beta1 the only real workaround here for Beta1 users? Just want to be clear on that before we relnote it.
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•