Bug 1334479 Show generic icon for legacy add-ons in post-install notification draft
authorAndrew Swan <aswan@mozilla.com>
Thu, 09 Feb 2017 15:20:02 -0800
changeset 483715 b6e807d31f71403b21e65dbf1da67d59666e13bb
parent 481324 88d6a5ed101805514831a01ed539cb916e214aa2
child 545695 cf57f775d59440fea9d9ed131ac94dad951f23e3
push id45374
push useraswan@mozilla.com
push dateTue, 14 Feb 2017 18:47:50 +0000
bugs1334479
milestone54.0a1
Bug 1334479 Show generic icon for legacy add-ons in post-install notification MozReview-Commit-ID: 9JsXPkqx909
browser/modules/ExtensionsUI.jsm
--- a/browser/modules/ExtensionsUI.jsm
+++ b/browser/modules/ExtensionsUI.jsm
@@ -355,20 +355,23 @@ this.ExtensionsUI = {
 
     return new Promise(resolve => {
       let action = {
         label: bundle.getString("addonPostInstall.okay.label"),
         accessKey: bundle.getString("addonPostInstall.okay.key"),
         callback: resolve,
       };
 
+      let icon = addon.isWebExtension ?
+                 addon.iconURL || DEFAULT_EXTENSION_ICON :
+                 "chrome://browser/skin/addons/addon-install-installed.svg";
       let options = {
         hideClose: true,
         timeout: Date.now() + 30000,
-        popupIconURL: addon.iconURL || DEFAULT_EXTENSION_ICON,
+        popupIconURL: icon,
         eventCallback(topic) {
           if (topic == "showing") {
             let doc = this.browser.ownerDocument;
             doc.getElementById("addon-installed-notification-header")
                .innerHTML = msg1;
             doc.getElementById("addon-installed-notification-message")
                .innerHTML = msg2;
           } else if (topic == "dismissed") {