Bug 910391 - Remove offline cache observer parameter from confirmInstall. r=fabrice

browser/modules/webappsUI.jsm

@@ -137,7 +137,7 @@ this.webappsUI = {
             localDir = app.appProfile.localDir;
           }
 
-          DOMApplicationRegistry.confirmInstall(aData, localDir, null,
+          DOMApplicationRegistry.confirmInstall(aData, localDir,
             (aManifest) => {
               Task.spawn(function() {
                 try {

dom/apps/src/Webapps.jsm

@@ -1130,7 +1130,7 @@ this.DOMApplicationRegistry = {
 
         if (manifest.appcache_path) {
           debug("appcache found");
-          this.startOfflineCacheDownload(manifest, app, null, null, isUpdate);
+          this.startOfflineCacheDownload(manifest, app, null, isUpdate);
         } else {
           // hosted app with no appcache, nothing to do, but we fire a
           // downloaded event
@@ -1280,10 +1280,7 @@ this.DOMApplicationRegistry = {
     }).bind(this));
   },
 
-  startOfflineCacheDownload: function startOfflineCacheDownload(aManifest, aApp,
+  startOfflineCacheDownload: function(aManifest, aApp, aProfileDir, aIsUpdate) {
-                                                                aProfileDir,
-                                                                aOfflineCacheObserver,
-                                                                aIsUpdate) {
     if (!aManifest.appcache_path) {
       return;
     }
@@ -1320,9 +1317,6 @@ this.DOMApplicationRegistry = {
       AppDownloadManager.add(aApp.manifestURL, download);
 
       cacheUpdate.addObserver(new AppcacheObserver(aApp), false);
-      if (aOfflineCacheObserver) {
-        cacheUpdate.addObserver(aOfflineCacheObserver, false);
-      }
     }).bind(this));
   },
 
@@ -1928,8 +1922,7 @@ this.DOMApplicationRegistry = {
     if (cacheDownload) {
       this.startOfflineCacheDownload(cacheDownload.manifest,
                                      cacheDownload.app,
-                                     cacheDownload.profileDir,
+                                     cacheDownload.profileDir);
-                                     cacheDownload.offlineCacheObserver);
       delete this.queuedDownload[aManifestURL];
 
       return;
@@ -1987,8 +1980,7 @@ this.DOMApplicationRegistry = {
     }
   },
 
-  confirmInstall: function(aData, aProfileDir, aOfflineCacheObserver,
+  confirmInstall: function(aData, aProfileDir, aInstallSuccessCallback) {
-                           aInstallSuccessCallback) {
     let isReinstall = false;
     let app = aData.app;
     app.removable = true;
@@ -2086,8 +2078,7 @@ this.DOMApplicationRegistry = {
       this.queuedDownload[app.manifestURL] = {
         manifest: manifest,
         app: appObject,
-        profileDir: aProfileDir,
+        profileDir: aProfileDir
-        offlineCacheObserver: aOfflineCacheObserver
       }
     }
 

mobile/android/chrome/content/browser.js

@@ -6807,7 +6807,7 @@ var WebappsUI = {
         file.initWithPath(profilePath);
 
         let self = this;
-        DOMApplicationRegistry.confirmInstall(aData, file, null,
+        DOMApplicationRegistry.confirmInstall(aData, file,
           function (aManifest) {
             let localeManifest = new ManifestHelper(aManifest, aData.app.origin);
 

webapprt/WebappsHandler.jsm

@@ -67,7 +67,7 @@ this.WebappsHandler = {
           localDir = shell.appProfile.localDir;
         }
 
-        DOMApplicationRegistry.confirmInstall(data, localDir, null,
+        DOMApplicationRegistry.confirmInstall(data, localDir,
           function (aManifest) {
             WebappsInstaller.install(data, aManifest);
           }