Web Inspector: Remove line highlight on primary execution line while stepping because...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Oct 2016 21:24:35 +0000 (21:24 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Oct 2016 21:24:35 +0000 (21:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=163294
<rdar://problem/28721176>

Patch by Joseph Pecoraro <pecoraro@apple.com> 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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@207165 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

index 9899231..31b5ca6 100644 (file)
@@ -1,3 +1,15 @@
+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>
 
         Web Inspector: Revealed line not highlighted in TextEditor while debugger paused
index 9b0464a..0d033f6 100644 (file)
@@ -1220,6 +1220,7 @@ WebInspector.TextEditor = class TextEditor extends WebInspector.View
             if (this._executionLineHandle) {
                 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);
             }
         });
     }