Bug 1007770 - Use the app (i.e. "rocket ship") icon in the update notification. r=blassey, r=ibarlow, a=lsblakk
authorMyk Melez <myk@mozilla.org>
Thu, 22 May 2014 17:35:21 -0700
changeset 192406 35ec38e5d96ea024a11ad200b0958c4a4475da9c
parent 192405 3a0a14c737bc4e3d460908307f8c824049d35a5c
child 192407 d24783ed4ab50d4684f5167366183be2c94778b3
push id3605
push userryanvm@gmail.com
push dateWed, 28 May 2014 21:06:58 +0000
treeherdermozilla-beta@aef210a74b69 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersblassey, ibarlow, lsblakk
bugs1007770
milestone30.0
Bug 1007770 - Use the app (i.e. "rocket ship") icon in the update notification. r=blassey, r=ibarlow, a=lsblakk
mobile/android/modules/WebappManager.jsm
--- a/mobile/android/modules/WebappManager.jsm
+++ b/mobile/android/modules/WebappManager.jsm
@@ -311,17 +311,17 @@ this.WebappManager = {
         return;
       }
 
       let names = [manifestUrlToApp[url].name for (url of outdatedApps)].join(", ");
       let accepted = yield this._notify({
         title: PluralForm.get(outdatedApps.length, Strings.GetStringFromName("downloadUpdateTitle")).
                replace("#1", outdatedApps.length),
         message: Strings.formatStringFromName("downloadUpdateMessage", [names], 1),
-        icon: "drawable://alert_download",
+        icon: "drawable://alert_app",
       }).dismissed;
 
       if (accepted) {
         yield this._updateApks([manifestUrlToApp[url] for (url of outdatedApps)]);
       }
     }
     // There isn't a catch block because we want the error to propagate through
     // the promise chain, so callers can receive it and choose to respond to it.