Bug 1057685 - Regression: Tweak Browser:Quit to maintain existing support for add-ons - part deux. r=wesj, a=lmandel
authorMark Capella <markcapella@twcny.rr.com>
Thu, 28 Aug 2014 22:00:47 -0400
changeset 217820 dbfd315972996f66e4e4df422c51653a3969bb18
parent 217819 7b4d4b3b759868c2af21c8fe3fe1248aac1c6896
child 217821 7b689c3657e4b359e5c0d20659c3c23707d697c1
push id515
push userraliiev@mozilla.com
push dateMon, 06 Oct 2014 12:51:51 +0000
treeherdermozilla-release@267c7a481bef [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerswesj, lmandel
bugs1057685
milestone33.0
Bug 1057685 - Regression: Tweak Browser:Quit to maintain existing support for add-ons - part deux. r=wesj, a=lmandel
mobile/android/chrome/content/browser.js
--- a/mobile/android/chrome/content/browser.js
+++ b/mobile/android/chrome/content/browser.js
@@ -1626,17 +1626,19 @@ var BrowserApp = {
           type: "Search:Keyword",
           identifier: engine.identifier,
           name: engine.name,
           query: query
         });
         break;
 
       case "Browser:Quit":
-        this.quit(aData ? JSON.parse(aData) : null);
+        // Add-ons like QuitNow and CleanQuit provide aData as an empty-string ("").
+        // Pass undefined to invoke the methods default parms.
+        this.quit(aData ? JSON.parse(aData) : undefined);
         break;
 
       case "SaveAs:PDF":
         this.saveAsPDF(browser);
         break;
 
       case "Preferences:Set":
         this.setPreferences(aData);