Commit 5d3c5784 authored by Yang Guo's avatar Yang Guo Committed by Commit Bot

[debug] remove ScriptBreakPointType from debug test api wrapper.

R=jgruber@chromium.org

Bug: v8:5530
Change-Id: I1680beaa665b6937df2e26d20cb69cc8577a21a9
Reviewed-on: https://chromium-review.googlesource.com/937203Reviewed-by: 's avatarJakob Gruber <jgruber@chromium.org>
Commit-Queue: Yang Guo <yangguo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#51558}
parent 9a561c82
......@@ -129,8 +129,7 @@ assertTrue(breakListenerCalled, "Break listener not called on breakpoint set by
function SetBreakpointInI1Script() {
var i_script = Debug.findScript(i1);
assertTrue(!!i_script, "invalid script for i1");
Debug.setScriptBreakPoint(Debug.ScriptBreakPointType.ScriptId,
i_script.id, 4);
Debug.setScriptBreakPointById(i_script.id, 4);
}
// Creates the eval script and tries to set the breakpoint.
......
......@@ -67,11 +67,6 @@ class DebugWrapper {
this.ExceptionBreak = { Caught : 0,
Uncaught: 1 };
// The different script break point types.
this.ScriptBreakPointType = { ScriptId: 0,
ScriptName: 1,
ScriptRegExp: 2 };
// Store the current script id so we can skip corresponding break events.
this.thisScriptId = %FunctionGetScriptId(receive);
......@@ -139,13 +134,6 @@ class DebugWrapper {
return this.setBreakPointAtLocation(scriptid, loc, opt_condition);
}
setScriptBreakPoint(type, scriptid, opt_line, opt_column, opt_condition) {
// Only sets by script id are supported for now.
assertEquals(this.ScriptBreakPointType.ScriptId, type);
return this.setScriptBreakPointById(scriptid, opt_line, opt_column,
opt_condition);
}
setScriptBreakPointById(scriptid, opt_line, opt_column, opt_condition) {
const loc = %ScriptLocationFromLine2(scriptid, opt_line, opt_column, 0);
return this.setBreakPointAtLocation(scriptid, loc, opt_condition);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment