From a13901fe5fbfaeebb1e3805d39422d0ce10627dd Mon Sep 17 00:00:00 2001 From: Marco Castelluccio Date: Tue, 30 Jul 2013 18:37:00 -0700 Subject: [PATCH] Bug 777517 - Webapp installation alert should launch app on click. r=myk --- browser/modules/webappsUI.jsm | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/browser/modules/webappsUI.jsm b/browser/modules/webappsUI.jsm index 2eecea7fc012..503bed39544f 100644 --- a/browser/modules/webappsUI.jsm +++ b/browser/modules/webappsUI.jsm @@ -114,7 +114,7 @@ this.webappsUI = { } DOMApplicationRegistry.confirmInstall(aData, false, localDir); - installationSuccessNotification(app, aWindow); + installationSuccessNotification(aData, app, aWindow); } else { DOMApplicationRegistry.denyInstall(aData); } @@ -140,7 +140,15 @@ this.webappsUI = { } } -function installationSuccessNotification(app, aWindow) { +function installationSuccessNotification(aData, app, aWindow) { + let launcher = { + observe: function(aSubject, aTopic) { + if (aTopic == "alertclickcallback") { + WebappOSUtils.launch(aData.app); + } + } + }; + let bundle = aWindow.gNavigatorBundle; if (("@mozilla.org/alerts-service;1" in Cc)) { @@ -152,7 +160,7 @@ function installationSuccessNotification(app, aWindow) { notifier.showAlertNotification(app.iconURI.spec, bundle.getString("webapps.install.success"), app.appNameAsFilename, - false, null, null); + true, null, launcher); } catch (ex) {} }