Bug 958809 - Don't put broadcasted data into the regular TabStateCache r=yoric a=lsblakk
authorTim Taubert <ttaubert@mozilla.com>
Sat, 11 Jan 2014 08:56:22 +0100
changeset 169264 c3c5bafe647a0938eefc259734c8a32dcf62fdce
parent 169263 fe8d463a6adf963d6064b14daa0915575475bcbf
child 169265 964de5dfb9ce7149a85c451a167889284d60bbf0
push id4960
push userttaubert@mozilla.com
push dateThu, 16 Jan 2014 12:20:23 +0000
treeherdermozilla-aurora@68f02b6be062 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersyoric, lsblakk
bugs958809
milestone28.0a2
Bug 958809 - Don't put broadcasted data into the regular TabStateCache r=yoric a=lsblakk From 082ebc3f0985d327c59158d38651d2a2a6a8ab24 Mon Sep 17 00:00:00 2001
browser/components/sessionstore/src/TabStateCache.jsm
--- a/browser/components/sessionstore/src/TabStateCache.jsm
+++ b/browser/components/sessionstore/src/TabStateCache.jsm
@@ -290,22 +290,18 @@ let TabStateCacheInternal = {
    *        The new data to be stored for the given |browser|.
    */
   updatePersistent: function (browser, newData) {
     let data = this._persistentData.get(browser) || {};
 
     for (let key of Object.keys(newData)) {
       let value = newData[key];
       if (value === null) {
-        // Remove the field if the value is null.
-        this.removeField(browser, key);
         delete data[key];
       } else {
-        // Update the field otherwise.
-        this.updateField(browser, key, value);
         data[key] = value;
       }
     }
 
     this._persistentData.set(browser, data);
   },
 
   _normalizeToBrowser: function(aKey) {