[Parser] Avoid on-the-fly internalization for natives_syntax.

[Parser] Avoid on-the-fly internalization for natives_syntax.

Rework Runtime::FunctionForName to take a c-string instead of a v8::String
so that the parser can parse native syntax runtime calls without doing
on-the-fly internalization.

Also adds a c-string variant of IntrinsicIndexForName for the same reasons.

BUG=v8:5215, chromium:634953
Committed: https://crrev.com/d15f45d4db96e6eb9fcdadc26be11f080845eea3
Cr-Commit-Position: refs/heads/master@{#39346}

[rmcilroy, 2016-09-08 14:39:49]

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

[commit-bot: I haz the power, 2016-09-08 14:39:57]

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

[rmcilroy, 2016-09-08 15:07:14]

Description was changed from

==========
[Parser] Avoid on-the-fly internalization for natives_syntax.

Rework Runtime::FunctionForName to take a c-string instead of a v8::String
so that the parser can parse native syntax runtime calls without doing
on-the-fly internalization.

Also adds a c-string variant of IntrinsicIndexForName for the same reasons.

BUG=V8:5215,chromium:634953
==========

to

==========
[Parser] Avoid on-the-fly internalization for natives_syntax.

Rework Runtime::FunctionForName to take a c-string instead of a v8::String
so that the parser can parse native syntax runtime calls without doing
on-the-fly internalization.

Also adds a c-string variant of IntrinsicIndexForName for the same reasons.

BUG=v8:5215,chromium:634953
==========

[commit-bot: I haz the power, 2016-09-08 15:10:11]

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

[commit-bot: I haz the power, 2016-09-08 15:10:12]

Dry run: This issue passed the CQ dry run.

[rmcilroy, 2016-09-08 16:27:48]

rmcilroy@chromium.org changed reviewers:
+ marja@chromium.org

[rmcilroy, 2016-09-08 16:27:49]

Marja, what do you think of this approach. This, along with Jochen's
"Deserialize the scope chain after parsing, not before" CL
(https://codereview.chromium.org/2306413002/) allows us to avoid on-the-fly
internalization during parsing entirely (WIP CL based on your original one at 
https://codereview.chromium.org/2328593002, currently fails two tests which are
the same two which fail in Jochen's CL on which it depends).

[marja, 2016-09-09 06:04:56]

lgtm, I think this is the right approach. (At least from the parser's point of
view.)

You'll still need a heap owner to review.

[rmcilroy, 2016-09-09 08:14:47]

rmcilroy@chromium.org changed reviewers:
+ ulan@chromium.org

[rmcilroy, 2016-09-09 08:14:48]

Ulan, PTAL at the heap changes, thanks.

[rmcilroy, 2016-09-12 08:47:11]

On 2016/09/09 08:14:48, rmcilroy wrote:
> Ulan, PTAL at the heap changes, thanks.

Ping, Ulan could you take a look at the heap changes please?

[ulan, 2016-09-12 10:50:25]

heap lgtm

[rmcilroy, 2016-09-12 10:58:07]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-09-12 10:58:12]

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-09-12 11:35:44]

Description was changed from

==========
[Parser] Avoid on-the-fly internalization for natives_syntax.

Rework Runtime::FunctionForName to take a c-string instead of a v8::String
so that the parser can parse native syntax runtime calls without doing
on-the-fly internalization.

Also adds a c-string variant of IntrinsicIndexForName for the same reasons.

BUG=v8:5215,chromium:634953
==========

to

==========
[Parser] Avoid on-the-fly internalization for natives_syntax.

Rework Runtime::FunctionForName to take a c-string instead of a v8::String
so that the parser can parse native syntax runtime calls without doing
on-the-fly internalization.

Also adds a c-string variant of IntrinsicIndexForName for the same reasons.

BUG=v8:5215,chromium:634953
==========

[commit-bot: I haz the power, 2016-09-12 11:35:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-12 11:36:07]

Description was changed from

==========
[Parser] Avoid on-the-fly internalization for natives_syntax.

Rework Runtime::FunctionForName to take a c-string instead of a v8::String
so that the parser can parse native syntax runtime calls without doing
on-the-fly internalization.

Also adds a c-string variant of IntrinsicIndexForName for the same reasons.

BUG=v8:5215,chromium:634953
==========

to

==========
[Parser] Avoid on-the-fly internalization for natives_syntax.

Rework Runtime::FunctionForName to take a c-string instead of a v8::String
so that the parser can parse native syntax runtime calls without doing
on-the-fly internalization.

Also adds a c-string variant of IntrinsicIndexForName for the same reasons.

BUG=v8:5215,chromium:634953

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

[commit-bot: I haz the power, 2016-09-12 11:36:08]

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