[debug-wrapper] migrate some scope related tests

[debug-wrapper] migrate some scope related tests

R=jgruber@chromium.org
BUG=v8:5530
Committed: https://crrev.com/341b39f9d0d891422c536e04cee7710d9ea8eeab
Cr-Commit-Position: refs/heads/master@{#41688}

[Yang, 2016-12-12 11:25:56]

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

[commit-bot: I haz the power, 2016-12-12 11:26:03]

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-12-12 11:50:03]

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

[commit-bot: I haz the power, 2016-12-12 11:50:04]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-12-12 13:11:58]

Thanks! lgtm with comments.

https://codereview.chromium.org/2566093002/diff/1/test/debugger/debug/debug-r...
File test/debugger/debug/debug-return-value.js (right):

https://codereview.chromium.org/2566093002/diff/1/test/debugger/debug/debug-r...
test/debugger/debug/debug-return-value.js:43:
//assertFalse(exec_state.frame(0).isAtReturn())
Can we remove this and below?

https://codereview.chromium.org/2566093002/diff/1/test/debugger/debug/debug-s...
File test/debugger/debug/debug-scopes.js (right):

https://codereview.chromium.org/2566093002/diff/1/test/debugger/debug/debug-s...
test/debugger/debug/debug-scopes.js:140: if (typeof(minimum_content[p]) ===
'function') {
We can remove the function branch as in the other files.

https://codereview.chromium.org/2566093002/diff/1/test/debugger/debugger.status
File test/debugger/debugger.status (right):

https://codereview.chromium.org/2566093002/diff/1/test/debugger/debugger.stat...
test/debugger/debugger.status:29: 'debug-scopes': [PASS, SLOW],
debug/debug-scopes

https://codereview.chromium.org/2566093002/diff/1/test/debugger/debugger.stat...
test/debugger/debugger.status:61: 'harmony/async-function-debug-scopes': [SKIP],
debug/

[Yang, 2016-12-14 06:50:21]

thanks. addressed comments.

[Yang, 2016-12-14 06:50:26]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-12-14 06:50:26]

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

[commit-bot: I haz the power, 2016-12-14 06:50:29]

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-12-14 07:20:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481698226197460,
"parent_rev": "9b40d4bcbefe088d6b136d48c3237c73e435abbf", "commit_rev":
"55265837b983b697523fb65efe58aa191ec65c5f"}

[commit-bot: I haz the power, 2016-12-14 07:20:17]

Description was changed from

==========
[debug-wrapper] migrate some scope related tests


R=jgruber@chromium.org
BUG=v8:5530
==========

to

==========
[debug-wrapper] migrate some scope related tests


R=jgruber@chromium.org
BUG=v8:5530

Review-Url: https://codereview.chromium.org/2566093002
==========

[commit-bot: I haz the power, 2016-12-14 07:20:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-14 07:20:38]

Description was changed from

==========
[debug-wrapper] migrate some scope related tests


R=jgruber@chromium.org
BUG=v8:5530

Review-Url: https://codereview.chromium.org/2566093002
==========

to

==========
[debug-wrapper] migrate some scope related tests

R=jgruber@chromium.org
BUG=v8:5530

Committed: https://crrev.com/341b39f9d0d891422c536e04cee7710d9ea8eeab
Cr-Commit-Position: refs/heads/master@{#41688}
==========

[commit-bot: I haz the power, 2016-12-14 07:20:39]

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