Closed Bug 362329 Opened 13 years ago Closed 13 years ago

Autodiscovery should not discover rel="alternate stylesheet" as a feed

Categories

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

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 3 alpha2

People

(Reporter: lachlan.hunt, Assigned: sayrer)

References

()

Details

(Keywords: testcase)

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0

Web Apps 1 has clarified the definition of rel="alternate stylesheet" and the use of rel="alternate" for feed autodiscovery.  It states:

| if it does also contains the keyword stylesheet, the alternate keyword instead
| modifies the meaning of the stylesheet keyword in the way described for that
| keyword, and the rest of this subsection doesn't apply.

That means the definition for autodiscovery which is specified later in that subsection does not apply and a link like this should not be discovered.

<link rel="alternate stylesheet" type="application/atom+xml" href="/feed1" title="alternate stylesheet">

For further explanation, see
http://listserver.dreamhost.com/pipermail/whatwg-whatwg.org/2006-November/008054.html

This is also related to bug 362156.

Reproducible: Always

Steps to Reproduce:
1. Create a page with <link rel="alternate stylesheet" type="application/atom+xml" href="/feed1" title="alternate stylesheet">
2. Load it up and see if autodiscovery finds it.

(I tested this with the latest trunk as well)
Actual Results:  
The feed is discovered.

Expected Results:  
It shouldn't be.
Attached file Test case
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee: nobody → sayrer
Keywords: testcase
Target Milestone: --- → Firefox 3 alpha2
Severity: enhancement → normal
I think this is fine for now, but instead of regexs, we should probably split(" ") the rel value into hash keys if we ever get around to combining the onLinkAdded handlers.
Attachment #247558 - Flags: review?(gavin.sharp)
Attachment #247558 - Flags: review?(gavin.sharp) → review+
Checking in browser.js;
/cvsroot/mozilla/browser/base/content/browser.js,v  <--  browser.js
new revision: 1.743; previous revision: 1.742
done
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.