[runtime] Change MessageLocation::function to SFI.

[runtime] Change MessageLocation::function to SFI.

This changes the {MessageLocation} structure to no longer contain a
concrete {JSFunction} object but rather a {SharedFunctionInfo}. It is
much easier by now to determine, and also the concrete closure is never
actually being used.

R=yangguo@chromium.org

Review-Url: https://codereview.chromium.org/2628973005
Cr-Commit-Position: refs/heads/master@{#42324}
Committed: https://chromium.googlesource.com/v8/v8/+/4408f8f1d9a7bba8a705c9e0ae9198cb7cb32cd3

[Michael Starzinger, 2017-01-12 16:02:42]

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

[commit-bot: I haz the power, 2017-01-12 16:02:46]

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 16:49:40]

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

[commit-bot: I haz the power, 2017-01-12 16:49:41]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2017-01-13 10:50:13]

mstarzinger@chromium.org changed reviewers:
+ clemensh@chromium.org

[Michael Starzinger, 2017-01-13 10:50:14]

Yang: PTAL.
Clemens: FYI.

[Yang, 2017-01-13 10:52:31]

On 2017/01/13 10:50:14, Michael Starzinger wrote:
> Yang: PTAL.
> Clemens: FYI.

lgtm

[Clemens Hammacher, 2017-01-13 10:54:23]

On 2017/01/13 at 10:50:14, mstarzinger wrote:
> Yang: PTAL.
> Clemens: FYI.

fine for me, thanks.

[Michael Starzinger, 2017-01-13 10:55:05]

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

[commit-bot: I haz the power, 2017-01-13 10:55:09]

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 11:59:29]

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

[commit-bot: I haz the power, 2017-01-13 11:59:29]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2017-01-13 12:13:09]

The CQ bit was checked by mstarzinger@chromium.org

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

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

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484309589228170,
"parent_rev": "1ff4a8178e48becd54441a9a04ccea79e3074ed3", "commit_rev":
"4408f8f1d9a7bba8a705c9e0ae9198cb7cb32cd3"}

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

Description was changed from

==========
[runtime] Change MessageLocation::function to SFI.

This changes the {MessageLocation} structure to no longer contain a
concrete {JSFunction} object but rather a {SharedFunctionInfo}. It is
much easier by now to determine, and also the concrete closure is never
actually being used.

R=yangguo@chromium.org
==========

to

==========
[runtime] Change MessageLocation::function to SFI.

This changes the {MessageLocation} structure to no longer contain a
concrete {JSFunction} object but rather a {SharedFunctionInfo}. It is
much easier by now to determine, and also the concrete closure is never
actually being used.

R=yangguo@chromium.org

Review-Url: https://codereview.chromium.org/2628973005
Cr-Commit-Position: refs/heads/master@{#42324}
Committed:
https://chromium.googlesource.com/v8/v8/+/4408f8f1d9a7bba8a705c9e0ae9198cb7cb...
==========

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

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