All users were logged out of Bugzilla on October 13th, 2018

JavaScript strict warning: chrome://browser/content/browser.js, line 6597: reference to undefined property rels.feed

RESOLVED FIXED in Firefox 27

Status

()

RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: rnewman, Assigned: billm)

Tracking

Trunk
Firefox 27
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
for (let relVal in rels) {
      switch (relVal) {
        case "feed":
        case "alternate":
          if (!feedAdded) {
            if (!rels.feed && rels.alternate && rels.stylesheet)
              break;

            if (isValidFeed(link, link.ownerDocument.nodePrincipal, rels.feed)) {  <<<<<<< HERE
              FeedHandler.addFeed(link, link.ownerDocument);
              feedAdded = true;
            }
          }
          break;


On current trunk.
Created attachment 811450 [details] [diff] [review]
fix-rel-feed

This has been annoying me for a while.
Assignee: nobody → wmccloskey
Status: NEW → ASSIGNED
Attachment #811450 - Flags: review?(felipc)
Attachment #811450 - Flags: review?(felipc) → review+
Looks like browser/base/content/pageinfo/feeds.js needs fixing too?

Comment 4

5 years ago
https://hg.mozilla.org/mozilla-central/rev/1207ad888b05
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
You need to log in before you can comment on or make changes to this bug.