Closed Bug 655162 Opened 11 years ago Closed 11 years ago

A missing updates property in an update.rdf should be counted as no available updates not as a failure to check for updates


(Toolkit :: Add-ons Manager, defect)

Not set



Tracking Status
firefox5 - fixed


(Reporter: mossop, Assigned: mossop)




(1 file)

See bug 655143

We changed this in Firefox 4. It is mostly harmless but this was never considered an error case before so I guess there is no good reason why it should be now.
Depends on: 655143
Attached patch patch rev 1Splinter Review
This changes a missing updates property to mean no update but keeps an updates property that is something other than an RDF container as an error and tests both cases.
Attachment #530459 - Flags: review?(robert.bugzilla)
Whiteboard: [has patch][needs review rs]
not going to track this but we can talk about an approval when the patch is finished, reviewed and landed on m-c.
Attachment #530459 - Flags: review?(robert.bugzilla) → review+
Whiteboard: [has patch][needs review rs] → [has patch]
Landed yesterday:
Closed: 11 years ago
Flags: in-testsuite+
Flags: in-litmus-
Resolution: --- → FIXED
Whiteboard: [has patch]
Target Milestone: --- → mozilla6
Comment on attachment 530459 [details] [diff] [review]
patch rev 1

I'd like to land this before the last beta merge to make sure we take care of the update issue we discovered in bug 655143 one way or another (and for non-AMO add-ons as well). This is real low risk.
Attachment #530459 - Flags: approval-mozilla-aurora?
Now that AMO fixed the bug their side, users will only see the error from bug 655143 if they have add-ons installed from somewhere other than AMO and that update site is returning the same empty RDF that AMO used to. This is probably a small class of user but this patch should also be very safe.
I'm in favor of taking this fix for 5 if we are indeed intending to "push harder" on users to upgrade to our latest release. One less speedbump in the upgrade process, even for a small-ish group of users, would be really nice.
Attachment #530459 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #530459 - Flags: approval-mozilla-beta+
Verified fixed with:

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:7.0a1) Gecko/20110529 Firefox/7.0a1

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:6.0a2) Gecko/20110529 Firefox/6.0a2

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:5.0) Gecko/20100101 Firefox/5.0
You need to log in before you can comment on or make changes to this bug.