Changeset 207165 in webkit

Timestamp:

Oct 11, 2016 2:24:35 PM (8 years ago)

Author:

commit-queue@webkit.org

Message:

Web Inspector: Remove line highlight on primary execution line while stepping because it is distracting
​https://bugs.webkit.org/show_bug.cgi?id=163294
<rdar://problem/28721176>

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2016-10-11
Reviewed by Timothy Hatcher.

• UserInterface/Views/TextEditor.js:

(WebInspector.TextEditor.prototype._updateExecutionLine):
When setting the primary execution line, remove default line highlights.

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/TextEditor.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2016-10-11  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: Remove line highlight on primary execution line while stepping because it is distracting
+        https://bugs.webkit.org/show_bug.cgi?id=163294
+        <rdar://problem/28721176>
+
+        Reviewed by Timothy Hatcher.
+
+        * UserInterface/Views/TextEditor.js:
+        (WebInspector.TextEditor.prototype._updateExecutionLine):
+        When setting the primary execution line, remove default line highlights.
+
 2016-10-10  Matt Baker  <mattbaker@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/TextEditor.js

@@ -1221 +1221 @@
                 this._codeMirror.addLineClass(this._executionLineHandle, "wrap", WebInspector.TextEditor.ExecutionLineStyleClassName);
                 this._codeMirror.addLineClass(this._executionLineHandle, "wrap", "primary");
+                this._codeMirror.removeLineClass(this._executionLineHandle, "wrap", WebInspector.TextEditor.HighlightedStyleClassName);
             }
         });