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

Unified Diff: third_party/WebKit/LayoutTests/inspector-protocol/debugger/continueToLocation.html

Issue 2239473003: [DevTools] Removed interstatementLocation from Debugger.continueToLocation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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: third_party/WebKit/LayoutTests/inspector-protocol/debugger/continueToLocation.html
diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/debugger/continueToLocation.html b/third_party/WebKit/LayoutTests/inspector-protocol/debugger/continueToLocation.html
index dec791a62136110c302c61f43e09d3e07f013190..ce8d20c57f2b14eb955ce8336beaf102ead43ce8 100644
--- a/third_party/WebKit/LayoutTests/inspector-protocol/debugger/continueToLocation.html
+++ b/third_party/WebKit/LayoutTests/inspector-protocol/debugger/continueToLocation.html
@@ -7,13 +7,13 @@
function test()
{
var scenario = [
- // requested line number, insterstatement location, expected control parameter 'step', expected line number
- [ 8, false, 1, 8 ],
- [ 8, true, 1, 8 ],
- [ 12, false, 6, 17 ],
- [ 13, false, 6, 17 ],
- [ 17, false, 6, 17 ],
- [ 17, true, 6, 17 ],
+ // requested line number, expected control parameter 'step', expected line number
+ [ 8, 1, 8 ],
+ [ 8, 1, 8 ],
+ [ 12, 6, 17 ],
+ [ 13, 6, 17 ],
+ [ 17, 6, 17 ],
+ [ 17, 6, 17 ],
];
InspectorTest.sendCommand("Debugger.enable", {});
@@ -40,13 +40,13 @@ function test()
function nextScenarioStep(pos)
{
if (pos < scenario.length)
- gotoSinglePassChain(scriptId, scenario[pos][0], scenario[pos][1], scenario[pos][2], scenario[pos][3], nextScenarioStep.bind(this, pos + 1));
+ gotoSinglePassChain(scriptId, scenario[pos][0], scenario[pos][1], scenario[pos][2], nextScenarioStep.bind(this, pos + 1));
else
InspectorTest.completeTest();
}
}
- function gotoSinglePassChain(scriptId, lineNumber, interstatement, expectedResult, expectedLineNumber, next)
+ function gotoSinglePassChain(scriptId, lineNumber, expectedResult, expectedLineNumber, next)
{
InspectorTest.eventHandler["Debugger.paused"] = handleDebuggerPausedOne;
@@ -58,7 +58,7 @@ function test()
InspectorTest.eventHandler["Debugger.paused"] = handleDebuggerPausedTwo;
- InspectorTest.sendCommand("Debugger.continueToLocation", { location: { scriptId: scriptId, lineNumber: lineNumber, columnNumber: 0}, interstatementLocation: interstatement }, logContinueToLocation);
+ InspectorTest.sendCommand("Debugger.continueToLocation", { location: { scriptId: scriptId, lineNumber: lineNumber, columnNumber: 0} }, logContinueToLocation);
function logContinueToLocation(response)
{
@@ -70,7 +70,7 @@ function test()
}
function handleDebuggerPausedTwo(messageObject)
{
- InspectorTest.log("Paused after continueToLocation (insterstatement=" + interstatement + ")");
+ InspectorTest.log("Paused after continueToLocation");
var actualLineNumber = messageObject.params.callFrames[0].location.lineNumber;
InspectorTest.log("Stopped on line " + actualLineNumber + ", expected " + expectedLineNumber + ", requested " + lineNumber + ", (0-based numbers).");

Powered by Google App Engine
This is Rietveld 408576698