does TB need to have the full RDF parser/service for RSS1 feeds
Categories
(MailNews Core :: Feed Reader, enhancement)
Tracking
(Not tracked)
People
(Reporter: aceman, Unassigned)
References
Details
The parseAsRSS1() function at
https://searchfox.org/comm-central/source/mailnews/extensions/newsblog/content/feed-parser.js#319 seem to use nsIRDFDataSource and nsIRDFXMLParser.
Could that be converted to some other generic XML parser so that we can drop code from rdf/* at some point?
Comment 1•5 years ago
|
||
Definitely needs to be reworked, as this is now the last js consumer blocking rdf removal.
Not sure there's anything to grab from there or not, but the Firefox feed parsing removal was bug 1503214. The automated tests they had could probably be useful to resurrect (but that would be for another bug).
RSS1.0 rdf feeds are still very much out there, and must continue to be supported. It's possible, perhaps likely, that an xml parser could do it. But it will take investigation. The issue is similar to the saxx removal, eg Bug 1514669, as far as finding a new way of doing it in order to remove the fork.
Removing rdf from feeds is more that just RSS1.0 though; the 2 feed databases are rdf files, and rdf resource is used for just about everything. It would be a significant renovation.
The strict answer here is no, we're not going to use any special rdf parser. Covered in Bug 1534163.
Description
•