[debug-wrapper] clearAllBreakPoints and several scripts functions

[debug-wrapper] clearAllBreakPoints and several scripts functions

This adds clearAllBreakPoints functionality (which requires tracking set
breakpoints internally), and several script-related functions that rely
on runtime functions.

BUG=v8:5530
Committed: https://crrev.com/2c8a4155aad5cae6fd1148d72663cfa2e68f9637
Cr-Commit-Position: refs/heads/master@{#41064}

[jgruber, 2016-11-16 10:22:15]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-16 10:22:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jgruber, 2016-11-16 10:41:14]

jgruber@chromium.org changed reviewers:
+ yangguo@chromium.org

[commit-bot: I haz the power, 2016-11-16 10:49:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-16 10:49:40]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-11-16 14:32:00]

lgtm

https://codereview.chromium.org/2508853003/diff/1/test/debugger/test-api.js
File test/debugger/test-api.js (right):

https://codereview.chromium.org/2508853003/diff/1/test/debugger/test-api.js#n...
test/debugger/test-api.js:175: }
do we want to clear the set afterwards?

https://codereview.chromium.org/2508853003/diff/1/test/debugger/test-api.js#n...
test/debugger/test-api.js:254: // is the script source for the script in which
the function is defined. If the
80-char limit

[jgruber, 2016-11-16 15:18:21]

https://codereview.chromium.org/2508853003/diff/1/test/debugger/test-api.js
File test/debugger/test-api.js (right):

https://codereview.chromium.org/2508853003/diff/1/test/debugger/test-api.js#n...
test/debugger/test-api.js:175: }
On 2016/11/16 14:31:59, Yang wrote:
> do we want to clear the set afterwards?

Sounds like a good idea.
Good catch :)

https://codereview.chromium.org/2508853003/diff/1/test/debugger/test-api.js#n...
test/debugger/test-api.js:254: // is the script source for the script in which
the function is defined. If the
On 2016/11/16 14:31:59, Yang wrote:
> 80-char limit

Done.

[jgruber, 2016-11-16 15:25:22]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-16 15:25:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-16 16:09:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-16 16:09:17]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-11-17 09:31:39]

The CQ bit was checked by jgruber@chromium.org

[jgruber, 2016-11-17 09:31:40]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2508853003/#ps20001
(title: "Address comments")

[commit-bot: I haz the power, 2016-11-17 09:31:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-17 09:33:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:37:44]

Description was changed from

==========
[debug-wrapper] clearAllBreakPoints and several scripts functions

This adds clearAllBreakPoints functionality (which requires tracking set
breakpoints internally), and several script-related functions that rely
on runtime functions.

BUG=v8:5530
==========

to

==========
[debug-wrapper] clearAllBreakPoints and several scripts functions

This adds clearAllBreakPoints functionality (which requires tracking set
breakpoints internally), and several script-related functions that rely
on runtime functions.

BUG=v8:5530

Committed: https://crrev.com/2c8a4155aad5cae6fd1148d72663cfa2e68f9637
Cr-Commit-Position: refs/heads/master@{#41064}
==========

[commit-bot: I haz the power, 2016-11-17 22:37:45]

Patchset 2 (id:??) landed as
https://crrev.com/2c8a4155aad5cae6fd1148d72663cfa2e68f9637
Cr-Commit-Position: refs/heads/master@{#41064}