Oops. Apparently nobody involved in specifying RSS and Atom autodiscovery quite noticed the fact that since there's no name for a mime-type |type "/" subtype| production exclusive of "media-type" which is |type "/" subtype *( ";" parameter )|, we weren't actually saying that autodiscovery demands exactly |type="application/atom+xml"|, since |type="application/atom+xml; charset=utf"| is also a perfectly correct Atom media-type. Testcase and patch shortly.
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox1.5
Created attachment 197668 [details] [diff] [review] patch v.1 Also fixes an unnoticed bug in the harvestFeeds "hide duplicates and stuff" routine, which was getting the raw mime-type, rather than a case-normalized and trimmed version.
Attachment #197668 - Flags: review?(mconnor)
Attachment #197668 - Flags: review?(mconnor) → review+
Attachment #197668 - Flags: approval1.8b5?
can we get this landed on the trunk and verified there to make sure we don't break anything and that this solves the problem? After a trunk landing and verification we'll consider for the branch.
Checked in on trunk Checking in mozilla/browser/base/content/browser.js; /cvsroot/mozilla/browser/base/content/browser.js,v <-- browser.js new revision: 1.516; previous revision: 1.515 done
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
verified using Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20050930 Firefox/1.6a1. test case works like a charm.
Status: RESOLVED → VERIFIED
Attachment #197668 - Flags: approval1.8b5? → approval1.8b5+
1.8 branch: Checking in browser.js; /cvsroot/mozilla/browser/base/content/browser.js,v <-- browser.js new revision: 1.479.2.37; previous revision: 1.479.2.36 done
Whiteboard: [checkin needed][a+]
Resetting QA Contact to default.
QA Contact: nobody → rss.preview
You need to log in before you can comment on or make changes to this bug.