Title: [97065] trunk/Source/WebCore
Revision
97065
Author
[email protected]
Date
2011-10-10 09:40:44 -0700 (Mon, 10 Oct 2011)

Log Message

Web Inspector: Linkifier.reset throw an exception if it is called after reset of debuggerPresentationModel.
https://bugs.webkit.org/show_bug.cgi?id=69756

Reviewed by Yury Semikhatsky.

* inspector/front-end/DebuggerPresentationModel.js:
(WebInspector.DebuggerPresentationModel.Linkifier.prototype.reset):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (97064 => 97065)


--- trunk/Source/WebCore/ChangeLog	2011-10-10 16:39:01 UTC (rev 97064)
+++ trunk/Source/WebCore/ChangeLog	2011-10-10 16:40:44 UTC (rev 97065)
@@ -1,3 +1,13 @@
+2011-10-10  Ilya Tikhonovsky  <[email protected]>
+
+        Web Inspector: Linkifier.reset throw an exception if it is called after reset of debuggerPresentationModel.
+        https://bugs.webkit.org/show_bug.cgi?id=69756
+
+        Reviewed by Yury Semikhatsky.
+
+        * inspector/front-end/DebuggerPresentationModel.js:
+        (WebInspector.DebuggerPresentationModel.Linkifier.prototype.reset):
+
 2011-09-28  Pavel Podivilov  <[email protected]>
 
         Web Inspector: RawSourceCode could be mapped to multiple sources.

Modified: trunk/Source/WebCore/inspector/front-end/DebuggerPresentationModel.js (97064 => 97065)


--- trunk/Source/WebCore/inspector/front-end/DebuggerPresentationModel.js	2011-10-10 16:39:01 UTC (rev 97064)
+++ trunk/Source/WebCore/inspector/front-end/DebuggerPresentationModel.js	2011-10-10 16:40:44 UTC (rev 97065)
@@ -610,8 +610,10 @@
 
     reset: function()
     {
-        for (var id in this._anchorsForRawSourceCode)
-            this._model._rawSourceCode[id].removeEventListener(WebInspector.RawSourceCode.Events.SourceMappingUpdated, this._updateSourceAnchors, this);
+        for (var id in this._anchorsForRawSourceCode) {
+            if (this._model._rawSourceCode[id]) // In case of navigation the list of rawSourceCodes is empty.
+                this._model._rawSourceCode[id].removeEventListener(WebInspector.RawSourceCode.Events.SourceMappingUpdated, this._updateSourceAnchors, this);
+        }
         this._anchorsForRawSourceCode = {};
     },
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to