Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 365570 - FeedWriter doesn't work with handlers with chrome:// URI (throws exception when downloading favicon). r=gavin.

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@217593 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
mozilla.mano%sent.com
2007-01-02 00:33:43 +00:00
parent aee7f8b6a9
commit df7df362e0
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
mozilla/browser/components/feeds/src/FeedWriter.js
View File

@@ -715,7 +715,7 @@ FeedWriter.prototype = {
// For privacy reasons we cannot set the image attribute directly
// to the icon url, see Bug 358878
var uri = makeURI(handlers[i].uri);
if (uri)
if (uri && /^https?/.test(uri.scheme))
new iconDataURIGenerator(uri.prePath + "/favicon.ico", menuItem)
}
}

5
mozilla/browser/components/preferences/feeds.js
View File

@@ -187,8 +187,9 @@ var gFeedsPane = {
row.setAttribute("webhandlerurl", handlers[i].uri);
var uri = ios.newURI(handlers[i].uri, null, null);
row.setAttribute("image", uri.prePath + "/favicon.ico");
if (/^https?/.test(uri.scheme))
row.setAttribute("image", uri.prePath + "/favicon.ico");
readersList.appendChild(row);
}
},