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

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

Issue 2482513002: [inspector] use debuggerContext as current in getPossibleBreakpoints (Closed)
Patch Set: 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: test/inspector/debugger/get-possible-breakpoints.js
diff --git a/test/inspector/debugger/get-possible-breakpoints.js b/test/inspector/debugger/get-possible-breakpoints.js
index 3a2b3f2c416e80f16d2d66a5ae03f26bbd4868be..92211bae45f88e69f1a5ed848798c1a072da3e59 100644
--- a/test/inspector/debugger/get-possible-breakpoints.js
+++ b/test/inspector/debugger/get-possible-breakpoints.js
@@ -1,7 +1,6 @@
// Copyright 2016 the V8 project authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-// Flags: --expose-gc
print("Test for Debugger.getPossibleBreakpoints");
@@ -138,8 +137,14 @@ function foo6() { Promise.resolve().then(() => 42) }`;
.then(setAllBreakpoints)
.then(() => Protocol.Runtime.evaluate({ expression: "foo5(); foo6()"}))
.then(next);
- }
+ },
+ function getPossibleBreakpointsForFunctionWithConstArray(next) {
+ compileScript("function foo() { return []; }", { name: "with-offset.js", line_offset: 0, column_offset: 0 })
+ .then((scriptId) => Protocol.Debugger.getPossibleBreakpoints({ start: { lineNumber: 0, columnNumber: 0, scriptId: scriptId }}))
+ .then(InspectorTest.logMessage)
+ .then(next);
+ }
]);
function compileScript(source, origin) {
« no previous file with comments | « src/inspector/v8-debugger-agent-impl.cc ('k') | test/inspector/debugger/get-possible-breakpoints-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698