Force SharedFunctionInfo::name() to be a flat string

Force SharedFunctionInfo::name() to be a flat string

This fact is depended upon by, at least, Parser::ParseLazy, and quite
likely by other code. There was already code in %FunctionSetName
enforcing this invariant. This patch adds similar code to
Factory::NewSharedFunctionInfo().

BUG=v8:4659
LOG=n

Committed: https://crrev.com/58a9bc5b5346eed84e55361dd2557074f6108a5c
Cr-Commit-Position: refs/heads/master@{#33914}

[adamk, 2016-02-10 20:18:09]

adamk@chromium.org changed reviewers:
+ rossberg@chromium.org

[rossberg, 2016-02-11 07:59:25]

lgtm

[adamk, 2016-02-11 18:51:20]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-02-11 18:51:31]

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

[commit-bot: I haz the power, 2016-02-11 18:52:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-11 18:53:13]

Description was changed from

==========
Force SharedFunctionInfo::name() to be a flat string

This fact is depended upon by, at least, Parser::ParseLazy, and quite
likely by other code. There was already code in %FunctionSetName
enforcing this invariant. This patch adds similar code to
Factory::NewSharedFunctionInfo().

BUG=v8:4659
LOG=n
==========

to

==========
Force SharedFunctionInfo::name() to be a flat string

This fact is depended upon by, at least, Parser::ParseLazy, and quite
likely by other code. There was already code in %FunctionSetName
enforcing this invariant. This patch adds similar code to
Factory::NewSharedFunctionInfo().

BUG=v8:4659
LOG=n

Committed: https://crrev.com/58a9bc5b5346eed84e55361dd2557074f6108a5c
Cr-Commit-Position: refs/heads/master@{#33914}
==========

[commit-bot: I haz the power, 2016-02-11 18:53:14]

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