Remove fake feeds: support from feedhandlers

RESOLVED FIXED

Status

RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: alqahira, Assigned: alqahira)

Tracking

Details

Attachments

(1 attachment)

Created attachment 357621 [details] [diff] [review]
Remove fake feeds: support

As we've determined in the various feeds: bugs

1) feeds: is not, as we thought when writing the feedhandlers, someone's idea of a shorthand for feed:https://, and, therefore, our feedhandlers do not support handling real-world feeds: URIs, were one to encounter them;

2) When setting one of our feedhandlers as the default feed app, we do not set the default OS feeds: app to our feedhandler; and

3) We never emit feeds: URIs from the Camino UI.

Therefore, we have no reason to keep the bogus feeds: (non-)support in our feedhandlers, and removing it is also essentially a no-op.
Attachment #357621 - Flags: superreview?(stuart.morgan+bugzilla)

Comment 1

10 years ago
Comment on attachment 357621 [details] [diff] [review]
Remove fake feeds: support

sr=smorgan
Attachment #357621 - Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Landed on cvs trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.