Changeset 50840 in webkit

Timestamp:

Nov 11, 2009 1:37:08 PM (14 years ago)

Author:

pfeldman@chromium.org

Message:

2009-11-11 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Check that if Storage panel exists before calling its methods

​https://bugs.webkit.org/show_bug.cgi?id=31343

• inspector/front-end/inspector.js: (WebInspector.addDatabase): (WebInspector.addCookieDomain): (WebInspector.addDOMStorage): (WebInspector.updateDOMStorage):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/inspector.js (modified) (3 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-11-11  Pavel Feldman  <pfeldman@chromium.org>
+
+        Reviewed by Timothy Hatcher.
+
+        Check that if Storage panel exists before calling its methods
+
+        https://bugs.webkit.org/show_bug.cgi?id=31343
+
+        * inspector/front-end/inspector.js:
+        (WebInspector.addDatabase):
+        (WebInspector.addCookieDomain):
+        (WebInspector.addDOMStorage):
+        (WebInspector.updateDOMStorage):
+
 2009-11-11  Jens Alfke  <snej@chromium.org>
 

trunk/WebCore/inspector/front-end/inspector.js

@@ -1063 +1063 @@
 WebInspector.addDatabase = function(payload)
 {
+    if (!this.panels.storage)
+        return;
     var database = new WebInspector.Database(
         payload.id,
@@ -1073 +1075 @@
 WebInspector.addCookieDomain = function(domain)
 {
-    if (this.panels.storage)
-        this.panels.storage.addCookieDomain(domain);
+    if (!this.panels.storage)
+        return;
+    this.panels.storage.addCookieDomain(domain);
 }
 
 WebInspector.addDOMStorage = function(payload)
 {
+    if (!this.panels.storage)
+        return;
     var domStorage = new WebInspector.DOMStorage(
         payload.id,
@@ -1088 +1093 @@
 WebInspector.updateDOMStorage = function(storageId)
 {
+    if (!this.panels.storage)
+        return;
     this.panels.storage.updateDOMStorage(storageId);
 }