DevTools: Implement undo, redo operations for the DOMStorage views.

Source/devtools/front_end/DOMStorage.js

 /*
  * Copyright (C) 2008 Nokia Inc.  All rights reserved.
  * Copyright (C) 2013 Samsung Electronics. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 18 matching lines @@
 
 /**
  * @constructor
  * @param {string} securityOrigin
  * @param {boolean} isLocalStorage
  */
 WebInspector.DOMStorage = function(securityOrigin, isLocalStorage)
 {
     this._securityOrigin = securityOrigin;
     this._isLocalStorage = isLocalStorage;
+    this._storageHistory = new WebInspector.DOMStorageHistory(this);
 }
 
 /**
  * @param {string} securityOrigin
  * @param {boolean} isLocalStorage
  * @return {DOMStorageAgent.StorageId}
  */
 WebInspector.DOMStorage.storageId = function(securityOrigin, isLocalStorage)
 {
     return { securityOrigin: securityOrigin, isLocalStorage: isLocalStorage };
@@ skipping 26 matching lines @@
     {
         DOMStorageAgent.getDOMStorageItems(this.id, callback);
     },
 
     /**
      * @param {string} key
      * @param {string} value
      */
     setItem: function(key, value)
     {
-        DOMStorageAgent.setDOMStorageItem(this.id, key, value);
+        this._storageHistory.perform(new WebInspector.DOMStorageSetItemAction(this, key, value));
     },
 
     /**
      * @param {string} key
      */
     removeItem: function(key)
     {
-        DOMStorageAgent.removeDOMStorageItem(this.id, key);
+        this._storageHistory.perform(new WebInspector.DOMStorageRemoveItemAction(this, key));
+    },
+
+    undo: function()
+    {
+        this._storageHistory.undo();
+    },
+
+    redo: function()
+    {
+        this._storageHistory.redo();
     }
 }
 
+/**
+ * @constructor

[apavlov, 2013/07/31 12:51:35]

This should be @interface instead

+ * @param {WebInspector.DOMStorage} domStorage
+ */
+WebInspector.DOMStorageAction = function(domStorage)
+{
+    this._domStorage = domStorage;
+}
+
+WebInspector.DOMStorageAction.prototype = {
+    perform: function(callback)

[apavlov, 2013/07/31 12:51:35]

callback should be annotated using @param

+    {
+    },
+
+    undo: function()
+    {
+    },
+
+    redo: function()
+    {
+    }
+}
+
+/**
+ * @constructor
+ * @param {WebInspector.DOMStorage} domStorage

[apavlov, 2013/07/31 12:51:35]

also add the @implements annotation

+ * @param {string} key
+ * @extends {WebInspector.DOMStorageAction}

[apavlov, 2013/07/31 12:51:35]

this should be removed

+ */
+WebInspector.DOMStorageRemoveItemAction = function(domStorage, key)
+{
+    WebInspector.DOMStorageAction.call(this, domStorage);
+    this._key = key;
+}
+
+WebInspector.DOMStorageRemoveItemAction.prototype = {
+    __proto__: WebInspector.DOMStorageAction.prototype,

[apavlov, 2013/07/31 12:51:35]

We usually put these last in the prototype definition

+
+    perform: function(callback)
+    {
+        function valueReceived(error, exists, value)
+        {
+            this._exists = exists;
+            this._value = value;
+            this.redo();
+            callback();
+        }
+        DOMStorageAgent.getDOMStorageItem(this._domStorage.id, this._key, valueReceived.bind(this));
+    },
+
+    undo: function()
+    {
+        DOMStorageAgent.setDOMStorageItem(this._domStorage.id, this._key, this._value);
+    },
+
+    redo: function()
+    {
+        DOMStorageAgent.removeDOMStorageItem(this._domStorage.id, this._key);
+    }
+}
+
+/**
+ * @constructor

[apavlov, 2013/07/31 12:51:35]

ditto re @implements

+ * @param {WebInspector.DOMStorage} domStorage
+ * @param {string} key
+ * @extends {WebInspector.DOMStorageAction}
+ */
+WebInspector.DOMStorageSetItemAction = function(domStorage, key, value)
+{
+    WebInspector.DOMStorageAction.call(this, domStorage);
+    this._key = key;
+    this._value = value;
+}
+
+WebInspector.DOMStorageSetItemAction.prototype = {
+    __proto__: WebInspector.DOMStorageAction.prototype,
+
+    perform: function(callback)

[apavlov, 2013/07/31 12:51:35]

JsDoc (perhaps just @inherit)

+    {
+        function valueReceived(error, exists, value)
+        {
+            this._exists = exists;
+            this._oldValue = value;
+            this.redo();
+            callback();
+        }
+        DOMStorageAgent.getDOMStorageItem(this._domStorage.id, this._key, valueReceived.bind(this));
+    },
+
+    undo: function()
+    {
+        if (this._exists)
+            DOMStorageAgent.setDOMStorageItem(this._domStorage.id, this._key, this._oldValue);
+        else
+            DOMStorageAgent.removeDOMStorageItem(this._domStorage.id, this._key);
+    },
+
+    redo: function()
+    {
+        DOMStorageAgent.setDOMStorageItem(this._domStorage.id, this._key, this._value);
+    }
+}
+
+/**
+ * @constructor
+ */
+WebInspector.DOMStorageHistory = function(domStorage)

[apavlov, 2013/07/31 12:51:35]

JsDoc for domStorage

+{
+    this._domStorage = domStorage;
+    this._actions = [];
+    this._currentActionIndex = -1;
+}
+
+WebInspector.DOMStorageHistory.prototype = {
+    perform: function(action)

[apavlov, 2013/07/31 12:51:35]

JsDoc

+    {
+        if (!action)
+            return;
+
+        action.perform(function() {
+            this._actions.push(action);
+            ++this._currentActionIndex;
+        }.bind(this));
+    },
+
+    undo: function()
+    {
+        if (this._currentActionIndex < 0)
+            return;
+
+        var action = this._actions[this._currentActionIndex];
+        if (action)

[apavlov, 2013/07/31 12:51:35]

I believe you should console.assert(action) here, since no action at this point
is an error

+            action.undo();
+        --this._currentActionIndex;
+    },
+
+    redo: function()
+    {
+        if (this._currentActionIndex > this._actions.length)
+            return;
+
+        var action = this._actions[++this._currentActionIndex];
+        if (action)

[apavlov, 2013/07/31 12:51:35]

I believe you should console.assert(action) here, since no action at this point
is an error

+            action.redo();
+    }
+}
+
 /**
  * @constructor
  * @extends {WebInspector.Object}
  */
 WebInspector.DOMStorageModel = function()
 {
     this._storages = {};
     InspectorBackend.registerDOMStorageDispatcher(new WebInspector.DOMStorageDispatcher(this));
     DOMStorageAgent.enable();
     WebInspector.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.SecurityOriginAdded, this._securityOriginAdded, this);
@@ skipping 190 matching lines @@
     domStorageItemUpdated: function(storageId, key, oldValue, newValue)
     {
         this._model._domStorageItemUpdated(storageId, key, oldValue, newValue);
     },
 }
 
 /**
  * @type {WebInspector.DOMStorageModel}
  */
 WebInspector.domStorageModel = null;