Title: [112679] branches/chromium/1084/Source/WebCore/inspector/front-end/_javascript_SourceFrame.js
Revision
112679
Author
[email protected]
Date
2012-03-30 09:50:42 -0700 (Fri, 30 Mar 2012)

Log Message

Merge 112661 - Web Inspector: [Regression] Execution line is not revealed after pretty print.
https://bugs.webkit.org/show_bug.cgi?id=82727

Reviewed by Pavel Feldman.

This patch makes _javascript_SourceFrame reveal execution line after pretty print.

* inspector/front-end/_javascript_SourceFrame.js:
(WebInspector._javascript_SourceFrame.prototype.setExecutionLine):

[email protected]
BUG=121091
Review URL: https://chromiumcodereview.appspot.com/9950021

Modified Paths

Diff

Modified: branches/chromium/1084/Source/WebCore/inspector/front-end/_javascript_SourceFrame.js (112678 => 112679)


--- branches/chromium/1084/Source/WebCore/inspector/front-end/_javascript_SourceFrame.js	2012-03-30 16:47:35 UTC (rev 112678)
+++ branches/chromium/1084/Source/WebCore/inspector/front-end/_javascript_SourceFrame.js	2012-03-30 16:50:42 UTC (rev 112679)
@@ -429,8 +429,10 @@
     setExecutionLine: function(lineNumber)
     {
         this._executionLineNumber = lineNumber;
-        if (this.loaded)
+        if (this.loaded) {
             this.textViewer.addDecoration(lineNumber, "webkit-execution-line");
+            this.revealLine(this._executionLineNumber);
+        }
     },
 
     clearExecutionLine: function()
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to