[runtime] Change JavaScriptFrame::GetFunctions interface.

[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2626213002
Cr-Original-Commit-Position: refs/heads/master@{#42311}
Committed: https://chromium.googlesource.com/v8/v8/+/25a9364f25a40986f4d7266b1fe53a5921754f6a
Review-Url: https://codereview.chromium.org/2626213002
Cr-Commit-Position: refs/heads/master@{#42314}
Committed: https://chromium.googlesource.com/v8/v8/+/52d05e12188c21ccf1a30bafb1f545945dd28c3f

[Michael Starzinger, 2017-01-12 11:42:52]

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

[commit-bot: I haz the power, 2017-01-12 11:42:57]

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, 2017-01-12 12:13:12]

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

[commit-bot: I haz the power, 2017-01-12 12:13:12]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2017-01-12 14:49:41]

Description was changed from

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All all call sites requiring the former can
use the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org
==========

to

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org
==========

[Jarin, 2017-01-13 08:48:08]

lgtm. Nice!

[Michael Starzinger, 2017-01-13 09:14:47]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2017-01-13 09:14:54]

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, 2017-01-13 09:16:30]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484298887266160,
"parent_rev": "369e5c8cceb0d589fbb21f90a0bca2f27f0980c4", "commit_rev":
"25a9364f25a40986f4d7266b1fe53a5921754f6a"}

[commit-bot: I haz the power, 2017-01-13 09:16:36]

Description was changed from

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org
==========

to

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2626213002
Cr-Commit-Position: refs/heads/master@{#42311}
Committed:
https://chromium.googlesource.com/v8/v8/+/25a9364f25a40986f4d7266b1fe53a59217...
==========

[commit-bot: I haz the power, 2017-01-13 09:16:37]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/25a9364f25a40986f4d7266b1fe53a59217...

[Michael Starzinger, 2017-01-13 09:24:49]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2629113004/ by mstarzinger@chromium.org.

The reason for reverting is: Breaks compilation..

[Michael Starzinger, 2017-01-13 09:38:54]

Description was changed from

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2626213002
Cr-Commit-Position: refs/heads/master@{#42311}
Committed:
https://chromium.googlesource.com/v8/v8/+/25a9364f25a40986f4d7266b1fe53a59217...
==========

to

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2626213002
Cr-Commit-Position: refs/heads/master@{#42311}
Committed:
https://chromium.googlesource.com/v8/v8/+/25a9364f25a40986f4d7266b1fe53a59217...
==========

[Michael Starzinger, 2017-01-13 09:56:38]

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

[commit-bot: I haz the power, 2017-01-13 09:56:49]

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, 2017-01-13 10:27:04]

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

[commit-bot: I haz the power, 2017-01-13 10:27:05]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2017-01-13 10:27:28]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2017-01-13 10:27:28]

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

[commit-bot: I haz the power, 2017-01-13 10:27:35]

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, 2017-01-13 10:29:14]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484303248226840,
"parent_rev": "79df3f049e3c78e36aa5532d802ae8f95fdbed5b", "commit_rev":
"52d05e12188c21ccf1a30bafb1f545945dd28c3f"}

[commit-bot: I haz the power, 2017-01-13 10:29:18]

Description was changed from

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2626213002
Cr-Commit-Position: refs/heads/master@{#42311}
Committed:
https://chromium.googlesource.com/v8/v8/+/25a9364f25a40986f4d7266b1fe53a59217...
==========

to

==========
[runtime] Change JavaScriptFrame::GetFunctions interface.

This adapts the aformentioned interface to no longer return a list of
{JSFunction} objects, but {SharedFunctionInfo} objects instead. Since
deoptimization data only contains the latter as literals, this by now
represents the fast path. All call sites requiring the former can use
the slow path via {JavaScriptFrame::Summarize} instead.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2626213002
Cr-Original-Commit-Position: refs/heads/master@{#42311}
Committed:
https://chromium.googlesource.com/v8/v8/+/25a9364f25a40986f4d7266b1fe53a59217...
Review-Url: https://codereview.chromium.org/2626213002
Cr-Commit-Position: refs/heads/master@{#42314}
Committed:
https://chromium.googlesource.com/v8/v8/+/52d05e12188c21ccf1a30bafb1f545945dd...
==========

[commit-bot: I haz the power, 2017-01-13 10:29:19]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/v8/v8/+/52d05e12188c21ccf1a30bafb1f545945dd...