[debugger] add mixed-stack tests.

[debugger] add mixed-stack tests.

R=jgruber@chromium.org, mstarzinger@chromium.org
BUG=v8:5265
Committed: https://crrev.com/f03bebc77da30de2d15f52e482c015b94cd9000b
Cr-Commit-Position: refs/heads/master@{#38621}

[Yang, 2016-08-12 12:38:12]

On 2016/08/12 12:37:43, Yang wrote:

Note that these tests only work as intended with --ignition-staging and without
the activation finder.

[Yang, 2016-08-12 12:38:16]

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

[commit-bot: I haz the power, 2016-08-12 12:38:24]

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

[Michael Starzinger, 2016-08-12 12:49:08]

LGTM on the runtime functions.

[jgruber, 2016-08-12 12:51:34]

LGTM

https://codereview.chromium.org/2246483002/diff/1/test/mjsunit/ignition/debug...
File test/mjsunit/ignition/debug-step-mixed-stack.js (right):

https://codereview.chromium.org/2246483002/diff/1/test/mjsunit/ignition/debug...
test/mjsunit/ignition/debug-step-mixed-stack.js:5: // Flags: --expose-debug-as
debug --allow-natives-syntax
Would it be possible to add --ignition-staging here and in the other test?

https://codereview.chromium.org/2246483002/diff/1/test/mjsunit/ignition/debug...
test/mjsunit/ignition/debug-step-mixed-stack.js:11: var break_point;
This appears to be unused.

[commit-bot: I haz the power, 2016-08-12 13:10:05]

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

[commit-bot: I haz the power, 2016-08-12 13:10:06]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-08-12 13:27:23]

https://codereview.chromium.org/2246483002/diff/1/test/mjsunit/ignition/debug...
File test/mjsunit/ignition/debug-step-mixed-stack.js (right):

https://codereview.chromium.org/2246483002/diff/1/test/mjsunit/ignition/debug...
test/mjsunit/ignition/debug-step-mixed-stack.js:5: // Flags: --expose-debug-as
debug --allow-natives-syntax
On 2016/08/12 12:51:34, jgruber wrote:
> Would it be possible to add --ignition-staging here and in the other test?

We have test variants that explicitly include --ignition-staging, so I'd like to
not add it here.

https://codereview.chromium.org/2246483002/diff/1/test/mjsunit/ignition/debug...
test/mjsunit/ignition/debug-step-mixed-stack.js:11: var break_point;
On 2016/08/12 12:51:34, jgruber wrote:
> This appears to be unused.

Removed.

[Yang, 2016-08-12 13:27:27]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-08-12 13:27:28]

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

[commit-bot: I haz the power, 2016-08-12 13:27:36]

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-08-12 13:51:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-12 13:51:29]

Description was changed from

==========
[debugger] add mixed-stack tests.

R=jgruber@chromium.org, mstarzinger@chromium.org
BUG=v8:5265
==========

to

==========
[debugger] add mixed-stack tests.

R=jgruber@chromium.org, mstarzinger@chromium.org
BUG=v8:5265

Committed: https://crrev.com/f03bebc77da30de2d15f52e482c015b94cd9000b
Cr-Commit-Position: refs/heads/master@{#38621}
==========

[commit-bot: I haz the power, 2016-08-12 13:51:30]

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