[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 1 year and 8 months ago then 8 months ago Yang landed V8 patch: https://codereview.chromium.org/1216193002.
With this patch ExecutionState.frame report only not internal frames. We use this method for getting callframes in V8DebuggerImpl::currentCallFrames function.

BUG=595206
R=dgozman@chromium.org

Committed: https://crrev.com/6e21aab027a4c54a2763e05c52e3ad5956d36500
Cr-Commit-Position: refs/heads/master@{#382991}

[kozy, 2016-03-23 22:18:57]

Patchset #1 (id:1) has been deleted

[kozy, 2016-03-23 22:19:17]

Description was changed from

==========
[DevTools] Remove obsolete check from V8DebuggerImple::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 when this
method could be used for async call stacks.
Now we use this method only for synchronous stacks and it can't contain only
internal stack frames.

BUG=595206
R=dgozman@chromium.org
==========

to

==========
[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 when this
method could be used for async call stacks.
Now we use this method only for synchronous stacks and it can't contain only
internal stack frames.

BUG=595206
R=dgozman@chromium.org
==========

[kozy, 2016-03-23 22:19:27]

Dmitry, please take a look!

[kozy, 2016-03-23 23:02:37]

Description was changed from

==========
[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 when this
method could be used for async call stacks.
Now we use this method only for synchronous stacks and it can't contain only
internal stack frames.

BUG=595206
R=dgozman@chromium.org
==========

to

==========
[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 1 year
and 8 months ago then 8 months ago Yang landed V8 patch:
https://codereview.chromium.org/1216193002.
With this patch ExecutionState.frame report only not internal frames. We use
this method for getting callframes in V8DebuggerImpl::currentCallFrames
function.

BUG=595206
R=dgozman@chromium.org
==========

[kozy, 2016-03-23 23:03:01]

Updated description. Please take a look!

[dgozman, 2016-03-23 23:13:38]

lgtm

[kozy, 2016-03-23 23:14:07]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-03-23 23:14:23]

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

[commit-bot: I haz the power, 2016-03-24 00:34:29]

Description was changed from

==========
[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 1 year
and 8 months ago then 8 months ago Yang landed V8 patch:
https://codereview.chromium.org/1216193002.
With this patch ExecutionState.frame report only not internal frames. We use
this method for getting callframes in V8DebuggerImpl::currentCallFrames
function.

BUG=595206
R=dgozman@chromium.org
==========

to

==========
[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 1 year
and 8 months ago then 8 months ago Yang landed V8 patch:
https://codereview.chromium.org/1216193002.
With this patch ExecutionState.frame report only not internal frames. We use
this method for getting callframes in V8DebuggerImpl::currentCallFrames
function.

BUG=595206
R=dgozman@chromium.org
==========

[commit-bot: I haz the power, 2016-03-24 00:34:30]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-03-24 00:38:53]

Description was changed from

==========
[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 1 year
and 8 months ago then 8 months ago Yang landed V8 patch:
https://codereview.chromium.org/1216193002.
With this patch ExecutionState.frame report only not internal frames. We use
this method for getting callframes in V8DebuggerImpl::currentCallFrames
function.

BUG=595206
R=dgozman@chromium.org
==========

to

==========
[DevTools] Remove obsolete check from V8DebuggerImpl::currentCallFrames()

This check was introduced in https://codereview.chromium.org/374903002 1 year
and 8 months ago then 8 months ago Yang landed V8 patch:
https://codereview.chromium.org/1216193002.
With this patch ExecutionState.frame report only not internal frames. We use
this method for getting callframes in V8DebuggerImpl::currentCallFrames
function.

BUG=595206
R=dgozman@chromium.org

Committed: https://crrev.com/6e21aab027a4c54a2763e05c52e3ad5956d36500
Cr-Commit-Position: refs/heads/master@{#382991}
==========

[commit-bot: I haz the power, 2016-03-24 00:38:54]

Patchset 1 (id:??) landed as
https://crrev.com/6e21aab027a4c54a2763e05c52e3ad5956d36500
Cr-Commit-Position: refs/heads/master@{#382991}