Title: [98110] branches/chromium/912/Source/WebCore/inspector/front-end/ScriptsPanel.js
Revision
98110
Author
vse...@chromium.org
Date
2011-10-21 10:39:06 -0700 (Fri, 21 Oct 2011)

Log Message

Merge 97777 - Web Inspector: ScriptsPanel's should use similar logic to decide whether it could show an anchor location and actually showing it.
https://bugs.webkit.org/show_bug.cgi?id=70322

Reviewed by Pavel Feldman.

* inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype.canShowAnchorLocation):
(WebInspector.ScriptsPanel.prototype._showSourceLine):


BUG=101006
TBR=vse...@chromium.org
Review URL: http://codereview.chromium.org/8373002

Modified Paths

Diff

Modified: branches/chromium/912/Source/WebCore/inspector/front-end/ScriptsPanel.js (98109 => 98110)


--- branches/chromium/912/Source/WebCore/inspector/front-end/ScriptsPanel.js	2011-10-21 17:32:39 UTC (rev 98109)
+++ branches/chromium/912/Source/WebCore/inspector/front-end/ScriptsPanel.js	2011-10-21 17:39:06 UTC (rev 98110)
@@ -632,7 +632,7 @@
 
     canShowAnchorLocation: function(anchor)
     {
-        return this._debuggerEnabled && WebInspector.debuggerModel.scriptsForURL(anchor.href).length;
+        return this._debuggerEnabled && anchor.uiSourceCode;
     },
 
     showAnchorLocation: function(anchor)
@@ -643,7 +643,8 @@
     _showSourceLine: function(uiSourceCode, lineNumber)
     {
         var sourceFrame = this._showSourceFrameAndAddToHistory(uiSourceCode);
-        sourceFrame.highlightLine(lineNumber);
+        if (lineNumber)
+            sourceFrame.highlightLine(lineNumber);
     },
 
     _showSourceFrameAndAddToHistory: function(uiSourceCode)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to