1149274 - Clear site-permissions should clear all registered push notifications

Status: RESOLVED FIXED

(Core :: DOM: Push Subscriptions, defect)

Description

[Doug Turner (:dougt)]

Currently we're using webapps-clear-data, but that doesn't work for sites.



case "webapps-clear-data":
        debug("webapps-clear-data");

        let data = aSubject.QueryInterface(Ci.mozIApplicationClearPrivateDataParams);
        if (!data) {
          debug("webapps-clear-data: Failed to get information about application");
          return;
        }

        // TODO
        // Need a way to go from manifest url to 'all scopes registered for push in this app'
        let appsService = Cc["@mozilla.org/AppsService;1"]
                            .getService(Ci.nsIAppsService);
        let scope = appsService.getScopeByLocalId(data.appId);
        if (!scope) {
          debug("webapps-clear-data: No scope found for " + data.appId);
          return;
        }

        this._db.getByScope(scope, function(record) {
          this._db.delete(records.channelID, null, function() {
              debug("webapps-clear-data: " + scope +
                    " Could not delete entry " + records.channelID);

            // courtesy, but don't establish a connection
            // just for it
            if (this._ws) {
              debug("Had a connection, so telling the server");
              this._send("unregister", {channelID: records.channelID});
            }
          }.bind(this), function() {
            debug("webapps-clear-data: Error in getByScope(" + scope + ")");
          });
        });

        break;
    }

Comment 1

[Doug Turner (:dougt)]

Attachment: bug_xxxxxx_clearAll_from_ff_ui

Comment 2

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Comment on attachment 8594437 [details] [diff] [review]
bug_xxxxxx_clearAll_from_ff_ui

Review of attachment 8594437 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/push/PushService.jsm
@@ +1630,5 @@
>  
> +  _clearAll: function _clearAll() {
> +    return new Promise((resolve, reject) => {
> +      this._db.clearAll(() => resolve(),
> +                        () => reject("Database error"));

Why not just this._db.clearAll(resolve, reject)?

@@ +1633,5 @@
> +      this._db.clearAll(() => resolve(),
> +                        () => reject("Database error"));
> +    });
> +  },
> +  

Nit: trailing whitespace

Comment 3

[Doug Turner (:dougt)]

> Why not just this._db.clearAll(resolve, reject)?

I wanted the reject to have some error string.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/82ea538860bc

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/82ea538860bc