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

VERIFIED FIXED in Firefox 5

Status

()

VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: mossop, Assigned: mossop)

Tracking

Trunk
mozilla6
Points:
---
Bug Flags:
in-testsuite +
in-litmus -

Firefox Tracking Flags

(firefox5- fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
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.
(Assignee)

Updated

8 years ago
Depends on: 655143
(Assignee)

Updated

8 years ago
tracking-firefox5: --- → ?
(Assignee)

Comment 1

8 years ago
Created attachment 530459 [details] [diff] [review]
patch rev 1

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)
(Assignee)

Updated

8 years ago
Whiteboard: [has patch][needs review rs]

Comment 2

8 years ago
not going to track this but we can talk about an approval when the patch is finished, reviewed and landed on m-c.
tracking-firefox5: ? → -
Attachment #530459 - Flags: review?(robert.bugzilla) → review+
Whiteboard: [has patch][needs review rs] → [has patch]
(Assignee)

Comment 3

8 years ago
Landed yesterday: http://hg.mozilla.org/mozilla-central/rev/c6c669e643b5
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Flags: in-testsuite+
Flags: in-litmus-
Resolution: --- → FIXED
Whiteboard: [has patch]
Target Milestone: --- → mozilla6
(Assignee)

Comment 4

8 years ago
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?
(Assignee)

Comment 5

8 years ago
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.

Comment 6

8 years ago
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.

Updated

8 years ago
Attachment #530459 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Updated

8 years ago
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
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.