Bug 1401851 - Skip masked favicons in ContentLinkHandler until we support them. r=nanj

MozReview-Commit-ID: 99du7AD11x7

--HG--
extra : rebase_source : 488da59a8dcd3c233d9dda38ccb2535b787e9985
This commit is contained in:
Marco Bonardo 2017-09-21 10:02:38 +02:00
parent 5dff4808d1
commit 747ec544c2

View File

@ -298,8 +298,11 @@ this.ContentLinkHandler = {
case "apple-touch-icon":
case "apple-touch-icon-precomposed":
case "fluid-icon":
if (iconAdded || !Services.prefs.getBoolPref("browser.chrome.site_icons"))
if (link.hasAttribute("mask") || // Masked icons are not supported yet.
iconAdded ||
!Services.prefs.getBoolPref("browser.chrome.site_icons")) {
break;
}
iconAdded = handleFaviconLink(link, isRichIcon, chromeGlobal, faviconLoads);
break;