Title: [98103] trunk/Source/WebCore
Revision
98103
Author
podivi...@chromium.org
Date
2011-10-21 08:56:25 -0700 (Fri, 21 Oct 2011)

Log Message

Web Inspector: first line in file is not highlighted.
https://bugs.webkit.org/show_bug.cgi?id=70504

Reviewed by Pavel Feldman.

* inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame.prototype._initializeTextViewer):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (98102 => 98103)


--- trunk/Source/WebCore/ChangeLog	2011-10-21 15:43:03 UTC (rev 98102)
+++ trunk/Source/WebCore/ChangeLog	2011-10-21 15:56:25 UTC (rev 98103)
@@ -1,3 +1,13 @@
+2011-10-20  Pavel Podivilov  <podivi...@chromium.org>
+
+        Web Inspector: first line in file is not highlighted.
+        https://bugs.webkit.org/show_bug.cgi?id=70504
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/SourceFrame.js:
+        (WebInspector.SourceFrame.prototype._initializeTextViewer):
+
 2011-10-21  Andreas Kling  <kl...@webkit.org>
 
         Simplify CSSStyleRule::setSelectorText().

Modified: trunk/Source/WebCore/inspector/front-end/SourceFrame.js (98102 => 98103)


--- trunk/Source/WebCore/inspector/front-end/SourceFrame.js	2011-10-21 15:43:03 UTC (rev 98102)
+++ trunk/Source/WebCore/inspector/front-end/SourceFrame.js	2011-10-21 15:56:25 UTC (rev 98103)
@@ -310,7 +310,7 @@
         if (typeof this._executionLineNumber === "number")
             this.setExecutionLine(this._executionLineNumber);
 
-        if (this._lineToHighlight) {
+        if (typeof this._lineToHighlight === "number") {
             this.highlightLine(this._lineToHighlight);
             delete this._lineToHighlight;
         }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to