Bug 1245212 - Properly cancel SessionStore:update message timers if timeoutDisabled is set to true. r?billm draft
authorMike Conley <mconley@mozilla.com>
Mon, 01 Feb 2016 16:00:29 -0500
changeset 328293 03a6fc87b7557b5b95e3e4bbc4fff4639caf82fc
parent 327605 d07dbd40dcd209124149f331f60dd55c8da33fbe
child 513789 bc8b9e7318a16443a078aa8134857ce8edf20869
push id10332
push usermconley@mozilla.com
push dateTue, 02 Feb 2016 19:20:51 +0000
reviewersbillm
bugs1245212
milestone47.0a1
Bug 1245212 - Properly cancel SessionStore:update message timers if timeoutDisabled is set to true. r?billm
browser/components/sessionstore/content/content-sessionStore.js
--- a/browser/components/sessionstore/content/content-sessionStore.js
+++ b/browser/components/sessionstore/content/content-sessionStore.js
@@ -685,17 +685,17 @@ var MessageQueue = {
 
   /**
    * Disables sending batched messages on a timer. Also cancels any pending
    * timers.
    */
   set timeoutDisabled(val) {
     this._timeoutDisabled = val;
 
-    if (!val && this._timeout) {
+    if (val && this._timeout) {
       clearTimeout(this._timeout);
       this._timeout = null;
     }
 
     return val;
   },
 
   init() {