DevTools: Revoke undo/redo due to inconsistent behavior determining the source of storage mutation

Source/devtools/front_end/DOMStorage.js

Index: Source/devtools/front_end/DOMStorage.js
diff --git a/Source/devtools/front_end/DOMStorage.js b/Source/devtools/front_end/DOMStorage.js
index 6cc203b922206f47442f561b80638f302ff6845a..3ad5ce8b755b0ed679d0cc95dc427518a6b39d67 100644
--- a/Source/devtools/front_end/DOMStorage.js
+++ b/Source/devtools/front_end/DOMStorage.js
@@ -37,7 +37,6 @@ WebInspector.DOMStorage = function(securityOrigin, isLocalStorage)
 {
     this._securityOrigin = securityOrigin;
     this._isLocalStorage = isLocalStorage;
-    this._storageHistory = new WebInspector.DOMStorageHistory(this);
 }
 
 /**
@@ -91,7 +90,7 @@ WebInspector.DOMStorage.prototype = {
      */
     setItem: function(key, value)
     {
-        this._storageHistory.perform(new WebInspector.DOMStorageSetItemAction(this, key, value));
+        DOMStorageAgent.setDOMStorageItem(this.id, key, value);
     },
 
     /**
@@ -99,17 +98,7 @@ WebInspector.DOMStorage.prototype = {
      */
     removeItem: function(key)
     {
-        this._storageHistory.perform(new WebInspector.DOMStorageRemoveItemAction(this, key));
-    },
-
-    undo: function()
-    {
-        this._storageHistory.undo();
-    },
-
-    redo: function()
-    {
-        this._storageHistory.redo();
+        DOMStorageAgent.removeDOMStorageItem(this.id, key);
     },
 
     __proto__: WebInspector.Object.prototype
@@ -117,210 +106,6 @@ WebInspector.DOMStorage.prototype = {
 
 /**
  * @constructor
- * @param {WebInspector.DOMStorage} domStorage
- */
-WebInspector.DOMStorageAction = function(domStorage)
-{
-    this._domStorage = domStorage;
-}
-
-WebInspector.DOMStorageAction.prototype = {
-    /**
-     * @param {function()} callback
-     */
-    perform: function(callback)
-    {
-    },
-
-    undo: function()
-    {
-    },
-
-    redo: function()
-    {
-    }
-}
-
-/**
- * @constructor
- * @extends {WebInspector.DOMStorageAction}
- * @param {WebInspector.DOMStorage} domStorage
- * @param {string} key
- */
-WebInspector.DOMStorageRemoveItemAction = function(domStorage, key)
-{
-    WebInspector.DOMStorageAction.call(this, domStorage);
-    this._key = key;
-}
-
-WebInspector.DOMStorageRemoveItemAction.prototype = {
-    /**
-     * @override
-     */
-    perform: function(callback)
-    {
-        DOMStorageAgent.getValue(this._domStorage.id, this._key, valueReceived.bind(this));
-
-        /**
-         * @param {?Protocol.Error} error
-         * @param {string=} value
-         */
-        function valueReceived(error, value)
-        {
-            if (error)
-                return;
-
-            this._value = value;
-            this.redo();
-            callback();
-        }
-    },
-
-    /**
-     * @override
-     */
-    undo: function()
-    {
-        DOMStorageAgent.setDOMStorageItem(this._domStorage.id, this._key, this._value);
-    },
-
-    /**
-     * @override
-     */
-    redo: function()
-    {
-        DOMStorageAgent.removeDOMStorageItem(this._domStorage.id, this._key);
-    },
-
-    __proto__: WebInspector.DOMStorageAction.prototype
-}
-
-/**
- * @constructor
- * @extends {WebInspector.DOMStorageAction}
- * @param {WebInspector.DOMStorage} domStorage
- * @param {string} key
- * @param {string} value
- */
-WebInspector.DOMStorageSetItemAction = function(domStorage, key, value)
-{
-    WebInspector.DOMStorageAction.call(this, domStorage);
-    this._key = key;
-    this._value = value;
-}
-
-WebInspector.DOMStorageSetItemAction.prototype = {
-    /**
-     * @override
-     */
-    perform: function(callback)
-    {
-        DOMStorageAgent.getValue(this._domStorage.id, this._key, valueReceived.bind(this));
-
-        /**
-         * @param {?Protocol.Error} error
-         * @param {string=} value
-         */
-        function valueReceived(error, value)
-        {
-            if (error)
-                return;
-
-            if (typeof value === "undefined")
-                delete this._exists;
-            else {
-                this._exists = true;
-                this._oldValue = value;
-            }
-            this.redo();
-            callback();
-        }
-    },
-
-    /**
-     * @override
-     */
-    undo: function()
-    {
-        if (!this._exists)
-            DOMStorageAgent.removeDOMStorageItem(this._domStorage.id, this._key);
-        else
-            DOMStorageAgent.setDOMStorageItem(this._domStorage.id, this._key, this._oldValue);
-    },
-
-    /**
-     * @override
-     */
-    redo: function()
-    {
-        DOMStorageAgent.setDOMStorageItem(this._domStorage.id, this._key, this._value);
-    },
-
-    __proto__: WebInspector.DOMStorageAction.prototype
-}
-
-/**
- * @constructor
- * @param {WebInspector.DOMStorage} domStorage
- */
-WebInspector.DOMStorageHistory = function(domStorage)
-{
-    this._domStorage = domStorage;
-
-    /** @type {!Array.<!WebInspector.DOMStorageAction>} */
-    this._actions = [];
-    this._undoableActionIndex = -1;
-}
-
-WebInspector.DOMStorageHistory.MAX_UNDO_STACK_DEPTH = 256;
-
-WebInspector.DOMStorageHistory.prototype = {
-    /**
-     * @param {WebInspector.DOMStorageAction} action
-     */
-    perform: function(action)
-    {
-        if (!action)
-            return;
-
-        action.perform(actionCompleted.bind(this));
-        function actionCompleted()
-        {
-            if (this._undoableActionIndex + 1 === WebInspector.DOMStorageHistory.MAX_UNDO_STACK_DEPTH) {
-                this._actions.shift();
-                --this._undoableActionIndex;
-            } else if (this._undoableActionIndex + 1 < this._actions.length)
-                this._actions.splice(this._undoableActionIndex + 1);
-
-            this._actions.push(action);
-            ++this._undoableActionIndex;
-        }
-    },
-
-    undo: function()
-    {
-        if (this._undoableActionIndex < 0)
-            return;
-
-        var action = this._actions[this._undoableActionIndex];
-        console.assert(action);
-        action.undo();
-        --this._undoableActionIndex;
-    },
-
-    redo: function()
-    {
-        if (this._undoableActionIndex >= this._actions.length - 1)
-            return;
-
-        var action = this._actions[++this._undoableActionIndex];
-        console.assert(action);
-        action.redo();
-    }
-}
-
-/**
- * @constructor
  * @extends {WebInspector.Object}
  */
 WebInspector.DOMStorageModel = function()