Bug 1246632 - Fix ackMsgRecv in PushServiceHttp2.js. r=kitcambridge
authorDragana Damjanovic <dd.mozilla@gmail.com>
Mon, 08 Feb 2016 06:40:00 +0100
changeset 329820 3f31e2b533d3c592b579ecddad4c2eb48f58924a
parent 329819 86c439c29ae3c65c414d65a646d26e086397431d
child 329821 0a47d0927f3d0159a98d101dc61539796eeff023
push id10611
push userdmitchell@mozilla.com
push dateTue, 09 Feb 2016 15:15:43 +0000
reviewerskitcambridge
bugs1246632
milestone47.0a1
Bug 1246632 - Fix ackMsgRecv in PushServiceHttp2.js. r=kitcambridge
dom/push/PushServiceHttp2.jsm
--- a/dom/push/PushServiceHttp2.jsm
+++ b/dom/push/PushServiceHttp2.jsm
@@ -548,19 +548,17 @@ this.PushServiceHttp2 = {
     this._conns[aSubscriptionUri].lastStartListening = Date.now();
     this._conns[aSubscriptionUri].channel = conn.channel;
     this._conns[aSubscriptionUri].listener = conn.listener;
 
   },
 
   _ackMsgRecv: function(aAckUri) {
     console.debug("ackMsgRecv()", aAckUri);
-    // We can't do anything about it if it fails,
-    // so we don't listen for response.
-    this._deleteResource(aAckUri);
+    return this._deleteResource(aAckUri);
   },
 
   init: function(aOptions, aMainPushService, aServerURL) {
     console.debug("init()");
     this._mainPushService = aMainPushService;
     this._serverURI = aServerURL;
 
     return Promise.resolve();