Bug 1933158 - Prevent mask icons from being stored. r=mak

Differential Revision: https://phabricator.services.mozilla.com/D230809
2024-12-06 18:35:07 +00:00
parent 5ca9fcb235
commit dc9a3fb47b
2 changed files with 4 additions and 4 deletions
--- a/browser/actors/LinkHandlerChild.sys.mjs
+++ b/browser/actors/LinkHandlerChild.sys.mjs
@@ -112,8 +112,8 @@ export class LinkHandlerChild extends JSWindowActorChild {
          isRichIcon = true;
        // fall through
        case "icon":
-          if (iconAdded || link.hasAttribute("mask")) {
-            // Masked icons are not supported yet.
+          if (iconAdded || link.hasAttribute("color") || rel.includes("mask")) {
+            // TODO (Bug 1337397): Add support for mask-icon favicons.
            break;
          }

--- a/browser/base/content/test/favicons/file_mask_icon.html
+++ b/browser/base/content/test/favicons/file_mask_icon.html
@@ -3,8 +3,8 @@
  <head>
    <meta charset="UTF-8" />
    <title>Mask Icon</title>
-    <link rel="icon" mask href="moz.png" type="image/png" />
-    <link rel="mask-icon" href="moz.png" type="image/png" />
+    <link rel="shortcut icon" href="moz.png" type="image/png" color="#101010">
+    <link rel="icon mask-icon" href="moz.png" type="image/png" />
  </head>
  <body>
  </body>