Bug 958809 - Don't put broadcasted data into the regular TabStateCache r=yoric
authorTim Taubert <ttaubert@mozilla.com>
Sat, 11 Jan 2014 08:56:22 +0100
changeset 162982 ee80588ead207d49d962c890b7a8ce8b6571ce23
parent 162981 69a6c6c99c85ea48fcf1fb371e1f80b24acc7b49
child 162983 bd20b6aae4a68ffb769b5e9b722f6f52de1101e4
push id25977
push userttaubert@mozilla.com
push dateSun, 12 Jan 2014 09:26:17 +0000
treeherdermozilla-central@b029e8d3e130 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersyoric
bugs958809
milestone29.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 958809 - Don't put broadcasted data into the regular TabStateCache r=yoric From c4c69269516391beb36f362d76f3e440bb8cd50b 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) {