ValueSerializer: Share string encoding code with String and RegExp objects.

ValueSerializer: Share string encoding code with String and RegExp objects.

This avoids the need to pull in the UTF-8 encoding code from the public API,
and allows it to take advantage of any supported way that i::String can be
encoded (one- or two-byte).

Backward compatibility is maintained, but this is the behavior beginning
with this version.

BUG=chromium:686159
Review-Url: https://codereview.chromium.org/2665653004
Cr-Commit-Position: refs/heads/master@{#42872}
Committed: https://chromium.googlesource.com/v8/v8/+/591cc0b4ccdb6934d76ea163569b5194faef4dcf

[jbroman, 2017-01-30 19:43:49]

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

[commit-bot: I haz the power, 2017-01-30 19:43:58]

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-30 19:45:29]

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

[commit-bot: I haz the power, 2017-01-30 19:45:30]

Dry run: Try jobs failed on following builders:
  v8_linux_arm64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/33650)

[jbroman, 2017-01-31 15:38:36]

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

[commit-bot: I haz the power, 2017-01-31 15:38: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-31 16:04:49]

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

[commit-bot: I haz the power, 2017-01-31 16:04:50]

Dry run: This issue passed the CQ dry run.

[jbroman, 2017-01-31 17:53:06]

jbroman@chromium.org changed reviewers:
+ jkummerow@chromium.org

[jbroman, 2017-01-31 17:53:07]

This removes the need to convert to v8::Local<v8::String> to get UTF-8 text.

[Jakob Kummerow, 2017-02-01 20:29:57]

lgtm

[jbroman, 2017-02-01 21:57:03]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2017-02-01 21:57:09]

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-02-01 22:27:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485986223387290, "parent_rev":
"aa3422b6713040ace576f776b9373765e4e9806c", "commit_rev":
"591cc0b4ccdb6934d76ea163569b5194faef4dcf"}

[commit-bot: I haz the power, 2017-02-01 22:27:08]

Description was changed from

==========
ValueSerializer: Share string encoding code with String and RegExp objects.

This avoids the need to pull in the UTF-8 encoding code from the public API,
and allows it to take advantage of any supported way that i::String can be
encoded (one- or two-byte).

Backward compatibility is maintained, but this is the behavior beginning
with this version.

BUG=chromium:686159
==========

to

==========
ValueSerializer: Share string encoding code with String and RegExp objects.

This avoids the need to pull in the UTF-8 encoding code from the public API,
and allows it to take advantage of any supported way that i::String can be
encoded (one- or two-byte).

Backward compatibility is maintained, but this is the behavior beginning
with this version.

BUG=chromium:686159

Review-Url: https://codereview.chromium.org/2665653004
Cr-Commit-Position: refs/heads/master@{#42872}
Committed:
https://chromium.googlesource.com/v8/v8/+/591cc0b4ccdb6934d76ea163569b5194fae...
==========

[commit-bot: I haz the power, 2017-02-01 22:27:09]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/591cc0b4ccdb6934d76ea163569b5194fae...