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

Unified Diff: LayoutTests/inspector/sources/debugger/frameworks-steppings.html

Issue 424813004: DevTools: Don't stop debugger inside V8 internal scripts with empty stack trace. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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: LayoutTests/inspector/sources/debugger/frameworks-steppings.html
diff --git a/LayoutTests/inspector/sources/debugger/frameworks-steppings.html b/LayoutTests/inspector/sources/debugger/frameworks-steppings.html
index e0800784c1bf8db0e470e25711cd838cfc2aef7b..8cefa6cf45a9018ee48119e139bd18e2b6b503d1 100644
--- a/LayoutTests/inspector/sources/debugger/frameworks-steppings.html
+++ b/LayoutTests/inspector/sources/debugger/frameworks-steppings.html
@@ -50,55 +50,30 @@ function test()
function step1()
{
- InspectorTest.runTestFunctionAndWaitUntilPaused(didPause);
+ InspectorTest.runTestFunctionAndWaitUntilPaused(step2);
}
- var actions = [
- "Print", // debugger;
- "StepInto", "StepInto", "Print", // callback1
- "StepInto", "Print", // callback2
- "StepInto", "Print", // callback2, skipped
- "StepInto", "Print", // callback3
- "StepInto", "StepInto", "StepInto", "StepInto", "Print", // callback4
- "StepInto", "Print", // callback4, skipped
- "StepInto", "Print", // callback4, inside catch
- "StepOut", "Print", // return to callback3
- "StepOver", "Print", // return to callback2
- "StepInto", "Print", // return to callback1
- ];
-
- function didPause(callFrames, reason, breakpointIds, asyncStackTrace)
+ function step2()
{
- var action = actions.shift();
- if (action === "Print") {
- InspectorTest.captureStackTrace(callFrames);
- InspectorTest.addResult("");
- while (action === "Print")
- action = actions.shift();
- }
-
- if (!action) {
- InspectorTest.completeDebuggerTest();
- return;
- }
+ var actions = [
+ "Print", // debugger;
+ "StepInto", "StepInto", "Print", // callback1
+ "StepInto", "Print", // callback2
+ "StepInto", "Print", // callback2, skipped
+ "StepInto", "Print", // callback3
+ "StepInto", "StepInto", "StepInto", "StepInto", "Print", // callback4
+ "StepInto", "Print", // callback4, skipped
+ "StepInto", "Print", // callback4, inside catch
+ "StepOut", "Print", // return to callback3
+ "StepOver", "Print", // return to callback2
+ "StepInto", "Print", // return to callback1
+ ];
+ InspectorTest.waitUntilPausedAndPerformSteppingActions(actions, step3);
+ }
- InspectorTest.addResult("Executing " + action + "...");
- switch (action) {
- case "StepInto":
- WebInspector.panels.sources._stepIntoButton.element.click();
- break;
- case "StepOver":
- WebInspector.panels.sources._stepOverButton.element.click();
- break;
- case "StepOut":
- WebInspector.panels.sources._stepOutButton.element.click();
- break;
- default:
- InspectorTest.addResult("FAIL: Unknown action: " + action);
- InspectorTest.completeDebuggerTest();
- return;
- }
- InspectorTest.waitUntilResumed(InspectorTest.waitUntilPaused.bind(InspectorTest, didPause));
+ function step3()
+ {
+ InspectorTest.completeDebuggerTest();
}
}

Powered by Google App Engine
This is Rietveld 408576698