Bug 595094 - Implement Desktop Notifications in Fennec. Factor content permission prompt impl a bit. r=mfinkle

--HG--
extra : rebase_source : 069e872c6bd0d4121ed6dce9fc097264103af108

mobile/components/ContentPermissionPrompt.js

@@ -7,6 +7,28 @@ Cu.import("resource://gre/modules/Services.jsm");
 
 const kCountBeforeWeRemember = 5;
 
+function setPagePermission(type, uri, allow) {
+  let pm = Services.perms;
+  let contentPrefs = Services.contentPrefs;
+  let contentPrefName = type + ".request.remember";
+
+  if (!contentPrefs.hasPref(uri, contentPrefName))
+      contentPrefs.setPref(uri, contentPrefName, 0);
+
+  let count = contentPrefs.getPref(uri, contentPrefName);
+
+  if (allow == false)
+    count--;
+  else
+    count++;
+    
+  contentPrefs.setPref(uri, contentPrefName, count);
+  if (count == kCountBeforeWeRemember)
+    pm.add(uri, type, Ci.nsIPermissionManager.ALLOW_ACTION);
+  else if (count == -kCountBeforeWeRemember)
+    pm.add(uri, type, Ci.nsIPermissionManager.DENY_ACTION);
+}
+
 function ContentPermissionPrompt() {}
 
 ContentPermissionPrompt.prototype = {
@@ -14,97 +36,80 @@ ContentPermissionPrompt.prototype = {
 
   QueryInterface: XPCOMUtils.generateQI([Ci.nsIContentPermissionPrompt]),
 
-  prompt: function(aRequest) {
+  getChromeWindow: function getChromeWindow(aWindow) {
-    if (aRequest.type != "geolocation")
+     let chromeWin = aWindow.QueryInterface(Ci.nsIInterfaceRequestor)
-      return;
+                            .getInterface(Ci.nsIWebNavigation)
+                            .QueryInterface(Ci.nsIDocShellTreeItem)
+                            .rootTreeItem
+                            .QueryInterface(Ci.nsIInterfaceRequestor)
+                            .getInterface(Ci.nsIDOMWindow)
+                            .QueryInterface(Ci.nsIDOMChromeWindow);
+     return chromeWin;
+  },
+
+  getNotificationBoxForRequest: function getNotificationBoxForRequest(request) {
+    let notificationBox = null;
+    if (request.window) {
+      let requestingWindow = request.window.top;
+      let chromeWin = this.getChromeWindow(requestingWindow).wrappedJSObject;
+      return chromeWin.getNotificationBox(requestingWindow);
+    }
+
+    let chromeWin = request.element.ownerDocument.defaultView;
+    return chromeWin.Browser.getNotificationBox();
+  },
+
+  handleExistingPermission: function handleExistingPermission(request) {
+    let result = Services.perms.testExactPermission(request.uri, request.type);
+    if (result == Ci.nsIPermissionManager.ALLOW_ACTION) {
+      request.allow();
+      return true;
+    }
+    if (result == Ci.nsIPermissionManager.DENY_ACTION) {
+      request.cancel();
+      return true;
+    }
+    return false;
+  },
+
+  prompt: function(request) {
+    // returns true if the request was handled
+    if (this.handleExistingPermission(request))
+       return;
 
     let pm = Services.perms;
-    let result = pm.testExactPermission(aRequest.uri, "geo");
+    let notificationBox = this.getNotificationBoxForRequest(request);
-
+    let browserBundle = Services.strings.createBundle("chrome://browser/locale/browser.properties");
-    if (result == Ci.nsIPermissionManager.ALLOW_ACTION) {
+    
-      aRequest.allow();
+    let notification = notificationBox.getNotificationWithValue(request.type);
+    if (notification)
       return;
-    } else if (result == Ci.nsIPermissionManager.DENY_ACTION) {
-      aRequest.cancel();
-      return;
-    }
 
-    function setPagePermission(aUri, aAllow) {
+    let buttons = [{
-      let contentPrefs = Services.contentPrefs;
+      label: browserBundle.GetStringFromName(request.type + ".allow"),
+      accessKey: null,
+      callback: function(notification) {
+                setPagePermission(request.type, request.uri, true);
+                request.allow();},
+    },
+    {
+      label: browserBundle.GetStringFromName(request.type + ".dontAllow"),
+      accessKey: null,
+      callback: function(notification) {
+            setPagePermission(request.type, request.uri, false);
+            request.cancel();},
+    }];
 
-      if (!contentPrefs.hasPref(aRequest.uri, "geo.request.remember"))
+    let message = browserBundle.formatStringFromName(request.type + ".siteWantsTo",
-        contentPrefs.setPref(aRequest.uri, "geo.request.remember", 0);
+                                                     [request.uri.host], 1);
+    let newBar = notificationBox.appendNotification(message,
+                                                    request.type,
+                                                    "", // Notifications in Fennec do not display images.
+                                                    notificationBox.PRIORITY_WARNING_MEDIUM,
+                                                    buttons);
+    return;
+  },
 
-      let count = contentPrefs.getPref(aRequest.uri, "geo.request.remember");
-
-      if (aAllow == false)
-        count--;
-      else
-        count++;
-
-      contentPrefs.setPref(aRequest.uri, "geo.request.remember", count);
-
-      if (count == kCountBeforeWeRemember)
-        pm.add(aUri, "geo", Ci.nsIPermissionManager.ALLOW_ACTION);
-      else if (count == -kCountBeforeWeRemember)
-        pm.add(aUri, "geo", Ci.nsIPermissionManager.DENY_ACTION);
-    }
-
-    function getChromeWindow(aWindow) {
-      let chromeWin = aWindow
-        .QueryInterface(Ci.nsIInterfaceRequestor)
-        .getInterface(Ci.nsIWebNavigation)
-        .QueryInterface(Ci.nsIDocShellTreeItem)
-        .rootTreeItem
-        .QueryInterface(Ci.nsIInterfaceRequestor)
-        .getInterface(Ci.nsIDOMWindow)
-        .QueryInterface(Ci.nsIDOMChromeWindow);
-      return chromeWin;
-    }
-
-    let notificationBox = null;
-    if (aRequest.window) {
-      let requestingWindow = aRequest.window.top;
-      let chromeWin = getChromeWindow(requestingWindow).wrappedJSObject;
-      notificationBox = chromeWin.getNotificationBox(requestingWindow);
-    } else {
-      let chromeWin = aRequest.element.ownerDocument.defaultView;
-      notificationBox = chromeWin.Browser.getNotificationBox();
-    }
-
-    let notification = notificationBox.getNotificationWithValue("geolocation");
-    if (!notification) {
-      let browserBundle = Services.strings.createBundle("chrome://browser/locale/browser.properties");
-
-      let buttons = [{
-        label: browserBundle.GetStringFromName("geolocation.share"),
-        accessKey: null,
-        callback: function(notification) {
-          setPagePermission(aRequest.uri, true);
-          aRequest.allow();
-        },
-      },
-      {
-        label: browserBundle.GetStringFromName("geolocation.dontShare"),
-        accessKey: null,
-        callback: function(notification) {
-          setPagePermission(aRequest.uri, false);
-          aRequest.cancel();
-        },
-      }];
-
-      let message = browserBundle.formatStringFromName("geolocation.siteWantsToKnow",
-                                                       [aRequest.uri.host], 1);
-
-      let newBar = notificationBox.appendNotification(message,
-                                                      "geolocation",
-                                                      "chrome://browser/skin/images/geo-16.png",
-                                                      notificationBox.PRIORITY_WARNING_MEDIUM,
-                                                      buttons);
-      // Make this a geolocation notification.
-      newBar.setAttribute("type", "geo");
-    }
-  }
 };
 
 

mobile/components/MobileComponents.manifest

@@ -42,6 +42,7 @@ component {C6E8C44D-9F39-4AF7-BCC0-76E38A8310F5} ContentPermissionPrompt.js
 contract @mozilla.org/content-permission/prompt;1 {C6E8C44D-9F39-4AF7-BCC0-76E38A8310F5}
 
 # AlertsService.js
+
 component {fe33c107-82a4-41d6-8c64-5353267e04c9} AlertsService.js
 contract @mozilla.org/system-alerts-service;1 {fe33c107-82a4-41d6-8c64-5353267e04c9}
 

mobile/locales/en-US/chrome/browser.properties

@@ -122,9 +122,14 @@ identity.unknown.tooltip=This web site does not supply identity information.
 identity.ownerUnknown2=(unknown)
 
 # Geolocation UI
-geolocation.share=Share
+geolocation.allow=Share
-geolocation.dontShare=Don't share
+geolocation.dontAllow=Don't share
-geolocation.siteWantsToKnow=%S wants your location.
+geolocation.siteWantsTo=%S wants your location.
+
+# Desktop notification UI
+desktop-notification.allow=Allow
+desktop-notification.dontAllow=Don't allow
+desktop-notification.siteWantsTo=%S wants use notifications.
 
 # New Tab Popup
 # LOCALIZATION NOTE (newtabpopup): Semi-colon list of plural forms.

mobile/themes/core/jar.mn

@@ -140,4 +140,3 @@ chrome.jar:
   skin/images/task-close-40.png             (images/task-close-40.png)
   skin/images/task-back-40.png              (images/task-back-40.png)
   skin/images/task-back-rtl-40.png          (images/task-back-rtl-40.png)
-  skin/images/geo-16.png                    (images/geo-16.png)