Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(583)

Unified Diff: third_party/WebKit/Source/devtools/front_end/source_frame/SourcesTextEditor.js

Issue 2505413002: [DevTools] Prepare JavaScriptSourceFrame for multiple breakpoints per line (Closed)
Patch Set: addressed comments Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/source_frame/SourcesTextEditor.js
diff --git a/third_party/WebKit/Source/devtools/front_end/source_frame/SourcesTextEditor.js b/third_party/WebKit/Source/devtools/front_end/source_frame/SourcesTextEditor.js
index ae5b393a998a88d188729cfa2c31668c3bf4bb8b..8f88cd19fa21394158d2fc337bce97136126fb25 100644
--- a/third_party/WebKit/Source/devtools/front_end/source_frame/SourcesTextEditor.js
+++ b/third_party/WebKit/Source/devtools/front_end/source_frame/SourcesTextEditor.js
@@ -165,38 +165,6 @@ SourceFrame.SourcesTextEditor = class extends TextEditor.CodeMirrorTextEditor {
}
/**
- * @param {number} lineNumber
- * @param {boolean} disabled
- * @param {boolean} conditional
- */
- addBreakpoint(lineNumber, disabled, conditional) {
- if (lineNumber < 0 || lineNumber >= this.codeMirror().lineCount())
- return;
-
- var className = 'cm-breakpoint' + (conditional ? ' cm-breakpoint-conditional' : '') +
- (disabled ? ' cm-breakpoint-disabled' : '');
- this.codeMirror().addLineClass(lineNumber, 'wrap', className);
- }
-
- /**
- * @param {number} lineNumber
- */
- removeBreakpoint(lineNumber) {
- if (lineNumber < 0 || lineNumber >= this.codeMirror().lineCount())
- return;
-
- var wrapClasses = this.codeMirror().getLineHandle(lineNumber).wrapClass;
- if (!wrapClasses)
- return;
-
- var classes = wrapClasses.split(' ');
- for (var i = 0; i < classes.length; ++i) {
- if (classes[i].startsWith('cm-breakpoint'))
- this.codeMirror().removeLineClass(lineNumber, 'wrap', classes[i]);
- }
- }
-
- /**
* @param {string} type
* @param {boolean} leftToNumbers
*/

Powered by Google App Engine
This is Rietveld 408576698