2014-11-18 09:50:12 +00:00
|
|
|
// Copyright 2014 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.
|
|
|
|
|
|
|
|
|
|
|
|
Debug = debug.Debug
|
|
|
|
var exception = null;
|
|
|
|
var break_count = 0;
|
[debugger] tuned StepNext and StepOut at return position
Proposed behaviour:
- StepNext at return position go into next function call (no changes with current behavior, but implemented in v8::Debug instead of hack on inspector side);
- StepOut at return position go into next non-current function call.
We need this to have better stepping in cases with native functions, blackboxed functions and/or different embedder calls (e.g. event listeners).
New behavior could be illustrated with two examples (for more see stepping-with-natives-and-frameworks test):
- let's assume that we've blackboxed callAll function, this function just takes its arguments and call one after another:
var foo = () => 1;
callAll(foo, foo, () => 2);
If we break inside of first call of function foo. Then on..
..StepNext - we're able to reach second call of function foo,
..StepOut - we're able to reach () => 2 call.
- let's consider case with native function:
[1,2,3].map(x => x * 2)
If we break inside of first callback call, then with StepNext we can iterate through all calls of callback, with StepOut we go to next statement after .map call.
Implementation details:
- when we request break we schedule step-in function call for any step action at return position and for step-in at any position,
- when we request StepOut at return position - we mark current function as needed-to-be-ignored inside of PrepareStepIn(function) call,
- when we request StepOut at not return position - we set break at return position and ask debugger to just repeat last step action on next stepping-related break.
Design doc: https://docs.google.com/document/d/1ihXHOIhP_q-fJCA0e2EiXz_Zr3B08KMjaPifcaqZ60Q/edit
BUG=v8:6118,chromium:583193
R=dgozman@chromium.org,yangguo@chromium.org
Review-Url: https://codereview.chromium.org/2758483002
Cr-Commit-Position: refs/heads/master@{#44028}
2017-03-22 14:16:18 +00:00
|
|
|
const expected_breaks = 10;
|
2014-11-18 09:50:12 +00:00
|
|
|
|
|
|
|
function listener(event, exec_state, event_data, data) {
|
|
|
|
try {
|
|
|
|
if (event == Debug.DebugEvent.Break) {
|
|
|
|
assertTrue(exec_state.frameCount() != 0, "FAIL: Empty stack trace");
|
|
|
|
var source = exec_state.frame(0).sourceLineText();
|
|
|
|
print("paused at: " + source);
|
|
|
|
assertTrue(source.indexOf("// Break " + break_count + ".") > 0,
|
2014-11-19 15:16:30 +00:00
|
|
|
"Unexpected pause at: " + source + "\n" +
|
|
|
|
"Expected: // Break " + break_count + ".");
|
2014-11-18 09:50:12 +00:00
|
|
|
if (source.indexOf("StepOver.") !== -1) {
|
2021-06-01 08:32:39 +00:00
|
|
|
exec_state.prepareStep(Debug.StepAction.StepOver);
|
2016-11-23 12:38:04 +00:00
|
|
|
} else if (source.indexOf("StepOut.") !== -1) {
|
|
|
|
exec_state.prepareStep(Debug.StepAction.StepOut);
|
2014-11-18 09:50:12 +00:00
|
|
|
} else {
|
2021-06-01 08:32:39 +00:00
|
|
|
exec_state.prepareStep(Debug.StepAction.StepInto);
|
2014-11-18 09:50:12 +00:00
|
|
|
}
|
|
|
|
++break_count;
|
|
|
|
}
|
|
|
|
} catch (e) {
|
|
|
|
exception = e;
|
|
|
|
print(e, e.stack);
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
Debug.setListener(listener);
|
|
|
|
|
|
|
|
Promise.resolve(42)
|
|
|
|
.then(promise1)
|
|
|
|
.then(Object) // Should skip stepping into native.
|
|
|
|
.then(Boolean) // Should skip stepping into native.
|
|
|
|
.then(promise2)
|
2014-11-19 15:16:30 +00:00
|
|
|
.catch(promise3)
|
2016-04-22 09:01:36 +00:00
|
|
|
.then(promise4)
|
2014-11-18 09:50:12 +00:00
|
|
|
.catch(function(e) {
|
|
|
|
%AbortJS("FAIL: uncaught exception " + e);
|
|
|
|
});
|
|
|
|
|
2014-11-19 15:16:30 +00:00
|
|
|
function promise1() {
|
2014-11-18 09:50:12 +00:00
|
|
|
debugger; // Break 0.
|
2017-07-14 17:50:09 +00:00
|
|
|
return exception || 1 // Break 1.
|
|
|
|
; // Break 2. StepOver.
|
|
|
|
}
|
2014-11-18 09:50:12 +00:00
|
|
|
|
2014-11-19 15:16:30 +00:00
|
|
|
function promise2() {
|
2014-11-18 09:50:12 +00:00
|
|
|
throw new Error; // Break 3.
|
|
|
|
}
|
|
|
|
|
2014-11-19 15:16:30 +00:00
|
|
|
function promise3() {
|
2017-07-14 17:50:09 +00:00
|
|
|
return break_count // Break 4.
|
|
|
|
; // Break 5.
|
|
|
|
}
|
2014-11-19 15:16:30 +00:00
|
|
|
|
|
|
|
function promise4() {
|
2016-04-22 09:01:36 +00:00
|
|
|
finalize(); // Break 6. StepOver.
|
2017-07-14 17:50:09 +00:00
|
|
|
return 0 // Break 7.
|
|
|
|
; // Break 8. StepOut.
|
|
|
|
}
|
2014-11-19 15:16:30 +00:00
|
|
|
|
|
|
|
function finalize() {
|
2016-04-22 09:01:36 +00:00
|
|
|
Promise.resolve().then(function() {
|
[debugger] tuned StepNext and StepOut at return position
Proposed behaviour:
- StepNext at return position go into next function call (no changes with current behavior, but implemented in v8::Debug instead of hack on inspector side);
- StepOut at return position go into next non-current function call.
We need this to have better stepping in cases with native functions, blackboxed functions and/or different embedder calls (e.g. event listeners).
New behavior could be illustrated with two examples (for more see stepping-with-natives-and-frameworks test):
- let's assume that we've blackboxed callAll function, this function just takes its arguments and call one after another:
var foo = () => 1;
callAll(foo, foo, () => 2);
If we break inside of first call of function foo. Then on..
..StepNext - we're able to reach second call of function foo,
..StepOut - we're able to reach () => 2 call.
- let's consider case with native function:
[1,2,3].map(x => x * 2)
If we break inside of first callback call, then with StepNext we can iterate through all calls of callback, with StepOut we go to next statement after .map call.
Implementation details:
- when we request break we schedule step-in function call for any step action at return position and for step-in at any position,
- when we request StepOut at return position - we mark current function as needed-to-be-ignored inside of PrepareStepIn(function) call,
- when we request StepOut at not return position - we set break at return position and ask debugger to just repeat last step action on next stepping-related break.
Design doc: https://docs.google.com/document/d/1ihXHOIhP_q-fJCA0e2EiXz_Zr3B08KMjaPifcaqZ60Q/edit
BUG=v8:6118,chromium:583193
R=dgozman@chromium.org,yangguo@chromium.org
Review-Url: https://codereview.chromium.org/2758483002
Cr-Commit-Position: refs/heads/master@{#44028}
2017-03-22 14:16:18 +00:00
|
|
|
if (expected_breaks !== break_count) { // Break 9. StepOut.
|
2014-11-18 09:50:12 +00:00
|
|
|
%AbortJS("FAIL: expected <" + expected_breaks + "> breaks instead of <" +
|
|
|
|
break_count + ">");
|
|
|
|
}
|
|
|
|
if (exception !== null) {
|
|
|
|
%AbortJS("FAIL: exception: " + exception);
|
|
|
|
}
|
|
|
|
});
|
|
|
|
}
|