303252 - Feedview doesn't mark up .rdf file served as application/rdf+xml

Status: RESOLVED FIXED

(Firefox Graveyard :: RSS Discovery and Preview, defect)

Description

[Joe Drew (not getting mail)]

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8b4) Gecko/20050803 Firefox/1.0+

With the newly integrated feedview: for some reason, Feedview isn't marking up
this page. I don't have venkman installed, so I can't debug it, but it seems to
be of the form feedview wants. The only thing I can think of is that perhaps
application/rdf+xml (which is what the linked URL is served as) doesn't make the
document an instanceof XMLDocument.

Comment 1

[Jo Hermans]

This page uses <application/vnd.mozilla.xul+xml> as its mime-type (used for
XUL), not the rdf-type.

Comment 2

[Phil Ringnalda (:philor)]

No, the page doesn't serve .rdf as XUL's type: as comment 0 says, it's served as
application/rdf+xml (which is weird enough, since it's a non-RDF flavor of RSS).
But even pre-feedview, Page Info claims something served as application/rdf+xml
is XUL, which is either a clue to why feedview isn't getting them, or a bug in
Page Info.

Comment 3

[Phil Ringnalda (:philor)]

And indeed, it's a clue: see bug 256084

Comment 4

[Phil Ringnalda (:philor)]

Attachment: bandaid for RDF/XML == XUL weirdness

Easy enough to bandaid it, while we wait on bug 256084 - genuine XUL will just
load a millisecond slower while we check the root element, in exchange for not
getting one duplicate bug filed per RSS feed served as application/rdf+xml.

Comment 5

[:Gavin Sharp [email: [e-mail]]]

Checking in feedviewOverlay.js;
/cvsroot/mozilla/browser/components/feedview/content/feedviewOverlay.js,v  <--
feedviewOverlay.js
new revision: 1.2; previous revision: 1.1
done

Comment 6

[Phil Ringnalda (:philor)]

Regressed by bug 303848 - repatch coming in a minute.

Comment 7

[Phil Ringnalda (:philor)]

Attachment: fresh new bandaid

Comment 8

[:Gavin Sharp [email: [e-mail]]]

Checking in browser.js;
/cvsroot/mozilla/browser/base/content/browser.js,v  <--  browser.js
new revision: 1.476; previous revision: 1.475
done

Comment 9

[Tanner M. Young [:tmyoung]]

Resetting QA Contact to default.