[V8] Fixed infinite loop in Debug::PrepareStep

[V8] Fixed infinite loop in Debug::PrepareStep

frame->GetExpression always returns the same function. We should iterate through expression stack when we use Function.call.apply for finding actual target.

LOG=N
BUG=chromium:499479
R=yurys@chromium.org,yangguo@chromium.org

Committed: https://crrev.com/8e1c3a75b0acfed5baf79d120a26b263a56fcc28
Cr-Commit-Position: refs/heads/master@{#29048}

[kozy, 2015-06-15 16:32:17]

Yury, please take a look.

[yurys, 2015-06-16 10:37:15]

lgtm

https://codereview.chromium.org/1182703003/diff/1/test/cctest/test-debug.cc
File test/cctest/test-debug.cc (right):

https://codereview.chromium.org/1182703003/diff/1/test/cctest/test-debug.cc#n...
test/cctest/test-debug.cc:3839: "                Function.call.apply(bar);"
Can you add one more test for a deeper case, something like

Function.call.apply(Function.call, [Function.call, bar])?

[Yang, 2015-06-16 10:49:01]

On 2015/06/16 10:37:15, yurys wrote:
> lgtm
> 
> https://codereview.chromium.org/1182703003/diff/1/test/cctest/test-debug.cc
> File test/cctest/test-debug.cc (right):
> 
>
https://codereview.chromium.org/1182703003/diff/1/test/cctest/test-debug.cc#n...
> test/cctest/test-debug.cc:3839: "                Function.call.apply(bar);"
> Can you add one more test for a deeper case, something like
> 
> Function.call.apply(Function.call, [Function.call, bar])?

lgtm.

[kozy, 2015-06-16 10:49:58]

Yang, please take a look.

https://codereview.chromium.org/1182703003/diff/1/test/cctest/test-debug.cc
File test/cctest/test-debug.cc (right):

https://codereview.chromium.org/1182703003/diff/1/test/cctest/test-debug.cc#n...
test/cctest/test-debug.cc:3839: "                Function.call.apply(bar);"
On 2015/06/16 10:37:15, yurys wrote:
> Can you add one more test for a deeper case, something like
> 
> Function.call.apply(Function.call, [Function.call, bar])?

Done.

[kozy, 2015-06-16 10:51:31]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2015-06-16 10:51:31]

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

[commit-bot: I haz the power, 2015-06-16 10:51:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1182703003/20001

[commit-bot: I haz the power, 2015-06-16 11:09:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-16 11:10:15]

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