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

Unified Diff: test/inspector/debugger/get-possible-breakpoints-master.js

Issue 2728563002: [inspector] added type of break location into getPossibleBreakpoints output (Closed)
Patch Set: added DCHECK Created 3 years, 9 months 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: test/inspector/debugger/get-possible-breakpoints-master.js
diff --git a/test/inspector/debugger/get-possible-breakpoints-master.js b/test/inspector/debugger/get-possible-breakpoints-master.js
index f0edf00b0dd4a931208e4c042279b3df685d0028..5eadfd0cd64c49e58ac78a5400d9d15c5d996d05 100644
--- a/test/inspector/debugger/get-possible-breakpoints-master.js
+++ b/test/inspector/debugger/get-possible-breakpoints-master.js
@@ -25,10 +25,17 @@ function dumpAllLocations(message) {
});
for (var location of locations) {
var line = lines[location.lineNumber];
- line = line.slice(0, location.columnNumber) + '#' + line.slice(location.columnNumber);
+ line = line.slice(0, location.columnNumber) + locationMark(location.type) + line.slice(location.columnNumber);
lines[location.lineNumber] = line;
}
lines = lines.filter(line => line.indexOf('//# sourceURL=') === -1);
InspectorTest.log(lines.join('\n'));
return message;
}
+
+function locationMark(type) {
+ if (type === 'return') return '|R|';
+ if (type === 'call') return '|C|';
+ if (type === 'debuggerStatement') return '|D|';
+ return '|_|';
+}

Powered by Google App Engine
This is Rietveld 408576698