362156 - support the 'feed' link type in HTML

Status: RESOLVED FIXED

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

Description

[Robert Sayre]

A no-brainer.

Comment 1

[Robert Sayre]

Use the test case from bug 362239

https://bugzilla.mozilla.org/attachment.cgi?id=247042

Comment 2

[Robert Sayre]

Attachment: look for the feed relation

This adds support in <link>, but not <a>. Whoever ends up on microformat detail will have to handle that. 

I filed two spinoff bugs-

bug 362864: add support for feed discovery in anchor elements
bug 362863: decide what to do with feed mime types we don't understand

Comment 3

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

Comment on attachment 247561 [details] [diff] [review]
look for the feed relation

>RCS file: /cvsroot/mozilla/browser/base/content/browser.js,v

>+    for each (var relValue in erel.split(" ")) { 

Shouldn't this be .split(/\s/)?

Also, please remove any trailing whitespace that this patch adds.

Comment 4

[Robert Sayre]

Attachment: fix gavin's comments

Comment 5

[Robert Sayre]

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

Comment 6

[Dão Gottwald [:dao]]

(In reply to comment #4)
> Created an attachment (id=247854) [edit]

btw, rels.foo is just as good as rels["foo"].

Comment 7

[Peter van der Woude [:Peter6]]

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20061207 Minefield/3.0a1 ID:2006120714 [cairo]

verified/fixed