[cleanup] The %StringAdd runtime function is always called on Strings.

[cleanup] The %StringAdd runtime function is always called on Strings.

There's no point in trying to convert to a string, as this runtime
function must always be called with Strings, otherwise there's a bug.

R=yangguo@chromium.org

Review-Url: https://codereview.chromium.org/2748253004
Cr-Commit-Position: refs/heads/master@{#43830}
Committed: https://chromium.googlesource.com/v8/v8/+/ead696dc88add5d5416a273d9bccd8ef5d04d0ad

[Benedikt Meurer, 2017-03-15 12:10:53]

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

[commit-bot: I haz the power, 2017-03-15 12:10:59]

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

[Benedikt Meurer, 2017-03-15 12:22:49]

Hey Yang,

Here's some simple cleanup.
Please take a look.

Thanks,
Benedikt

[Benedikt Meurer, 2017-03-15 12:23:01]

Description was changed from

==========
[runtime] The %StringAdd runtime function is always called on Strings.

There's no point in trying to convert to a string, as this runtime
function must always be called with Strings, otherwise there's a bug.

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

to

==========
[cleanup] The %StringAdd runtime function is always called on Strings.

There's no point in trying to convert to a string, as this runtime
function must always be called with Strings, otherwise there's a bug.

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

[commit-bot: I haz the power, 2017-03-15 12:35:38]

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

[commit-bot: I haz the power, 2017-03-15 12:35:39]

Dry run: This issue passed the CQ dry run.

[Yang, 2017-03-15 13:28:12]

On 2017/03/15 12:22:49, Benedikt Meurer wrote:
> Hey Yang,
> 
> Here's some simple cleanup.
> Please take a look.
> 
> Thanks,
> Benedikt

lgtm

[Yang, 2017-03-15 13:28:14]

On 2017/03/15 12:22:49, Benedikt Meurer wrote:
> Hey Yang,
> 
> Here's some simple cleanup.
> Please take a look.
> 
> Thanks,
> Benedikt

lgtm

[Benedikt Meurer, 2017-03-15 13:28:54]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2017-03-15 13:29:06]

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-03-15 13:35:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489584534900230, "parent_rev":
"9fca3ad71bfbca9ef4c1517b8b4f8368edd68f61", "commit_rev":
"ead696dc88add5d5416a273d9bccd8ef5d04d0ad"}

[commit-bot: I haz the power, 2017-03-15 13:35:48]

Description was changed from

==========
[cleanup] The %StringAdd runtime function is always called on Strings.

There's no point in trying to convert to a string, as this runtime
function must always be called with Strings, otherwise there's a bug.

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

to

==========
[cleanup] The %StringAdd runtime function is always called on Strings.

There's no point in trying to convert to a string, as this runtime
function must always be called with Strings, otherwise there's a bug.

R=yangguo@chromium.org

Review-Url: https://codereview.chromium.org/2748253004
Cr-Commit-Position: refs/heads/master@{#43830}
Committed:
https://chromium.googlesource.com/v8/v8/+/ead696dc88add5d5416a273d9bccd8ef5d0...
==========

[commit-bot: I haz the power, 2017-03-15 13:35:49]

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