Changeset 92476 in webkit

Timestamp:

Aug 5, 2011 6:16:14 AM (13 years ago)

Author:

yurys@chromium.org

Message:

Web Inspector: check that detaching frame has been attached before removing it from the console selector
​https://bugs.webkit.org/show_bug.cgi?id=65686

Reviewed by Pavel Feldman.

• inspector/front-end/JavaScriptContextManager.js:

(WebInspector.JavaScriptContextManager.prototype._frameDetached):

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-08-05  Yury Semikhatsky  <yurys@chromium.org>
+
+        Web Inspector: check that detaching frame has been attached before removing it from the console selector
+        https://bugs.webkit.org/show_bug.cgi?id=65686
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/JavaScriptContextManager.js:
+        (WebInspector.JavaScriptContextManager.prototype._frameDetached):
+
 2011-08-05  Keishi Hattori  <keishi@webkit.org>
 

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

@@ -60 +60 @@
         var frameId = event.data;
         var context = this._frameIdToContext[frameId];
+        if (!context)
+            return;
         this._consoleView.removeContext(context);
         delete this._frameIdToContext[frameId];