Bug 877932 - Check if the app is locally installed in the checkInstalled function. r=fabrice
authorMarco Castelluccio <mar.castelluccio@studenti.unina.it>
Fri, 13 Sep 2013 08:07:04 -0400
changeset 160035 2981946d58f4b682a3a016a1af933c6fff7383ec
parent 160034 2aa1e16b67b05c67d451f99d4fd733d14351f3b3
child 160036 b3c433d488b9b829ab8827235d9e87f45f809a43
push id2961
push userlsblakk@mozilla.com
push dateMon, 28 Oct 2013 21:59:28 +0000
treeherdermozilla-beta@73ef4f13486f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfabrice
bugs877932
milestone26.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 877932 - Check if the app is locally installed in the checkInstalled function. r=fabrice
dom/apps/src/Webapps.jsm
--- a/dom/apps/src/Webapps.jsm
+++ b/dom/apps/src/Webapps.jsm
@@ -2938,17 +2938,18 @@ this.DOMApplicationRegistry = {
     }).bind(this));
   },
 
   checkInstalled: function(aData, aMm) {
     aData.app = null;
     let tmp = [];
 
     for (let appId in this.webapps) {
-      if (this.webapps[appId].manifestURL == aData.manifestURL) {
+      if (this.webapps[appId].manifestURL == aData.manifestURL &&
+          this._isLaunchable(this.webapps[appId])) {
         aData.app = AppsUtils.cloneAppObject(this.webapps[appId]);
         tmp.push({ id: appId });
         break;
       }
     }
 
     this._readManifests(tmp, (function(aResult) {
       for (let i = 0; i < aResult.length; i++) {