Revert of Remove StringBuilder::appendLiteral.

Revert of Remove StringBuilder::appendLiteral. (patchset #1 id:1 of https://codereview.chromium.org/2017053003/ )

Reason for revert:
Will unfortunately have to back out this one so that the revert https://codereview.chromium.org/2025503002/ will cleanly apply.

(Don't want a tree with this many ASan failures over the long weekend.)

Original issue's description:
> Remove StringBuilder::appendLiteral.
>
> We can just rely on the append(StringView) version instead. This does
> mean we have to take 2 branches in some cases, but it should be
> very minimal and since the code is inline the compiler can also
> likely eliminate it.
>
> BUG=615174
>
> Committed: https://crrev.com/04157e8b89881d033f9eeca4466d9dd0c4e9aaea
> Cr-Commit-Position: refs/heads/master@{#396601}

TBR=haraken@chromium.org,yutak@chromium.org,esprehn@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=615174
Committed: https://crrev.com/433566e878a7ce453ae7be38e0d252f8cf804411
Cr-Commit-Position: refs/heads/master@{#396640}

[sof, 2016-05-28 11:48:17]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-05-28 11:48:18]

Created Revert of Remove StringBuilder::appendLiteral.

[commit-bot: I haz the power, 2016-05-28 11:48:25]

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

[commit-bot: I haz the power, 2016-05-28 11:50:11]

Description was changed from

==========
Revert of Remove StringBuilder::appendLiteral. (patchset #1 id:1 of
https://codereview.chromium.org/2017053003/ )

Reason for revert:
Will unfortunately have to back out this one so that the revert
https://codereview.chromium.org/2025503002/ will cleanly apply.

(Don't want a tree with this many ASan failures over the long weekend.)

Original issue's description:
> Remove StringBuilder::appendLiteral.
> 
> We can just rely on the append(StringView) version instead. This does
> mean we have to take 2 branches in some cases, but it should be
> very minimal and since the code is inline the compiler can also
> likely eliminate it.
> 
> BUG=615174
> 
> Committed: https://crrev.com/04157e8b89881d033f9eeca4466d9dd0c4e9aaea
> Cr-Commit-Position: refs/heads/master@{#396601}

TBR=haraken@chromium.org,yutak@chromium.org,esprehn@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=615174
==========

to

==========
Revert of Remove StringBuilder::appendLiteral. (patchset #1 id:1 of
https://codereview.chromium.org/2017053003/ )

Reason for revert:
Will unfortunately have to back out this one so that the revert
https://codereview.chromium.org/2025503002/ will cleanly apply.

(Don't want a tree with this many ASan failures over the long weekend.)

Original issue's description:
> Remove StringBuilder::appendLiteral.
> 
> We can just rely on the append(StringView) version instead. This does
> mean we have to take 2 branches in some cases, but it should be
> very minimal and since the code is inline the compiler can also
> likely eliminate it.
> 
> BUG=615174
> 
> Committed: https://crrev.com/04157e8b89881d033f9eeca4466d9dd0c4e9aaea
> Cr-Commit-Position: refs/heads/master@{#396601}

TBR=haraken@chromium.org,yutak@chromium.org,esprehn@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=615174
==========

[commit-bot: I haz the power, 2016-05-28 11:50:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-28 11:51:25]

Description was changed from

==========
Revert of Remove StringBuilder::appendLiteral. (patchset #1 id:1 of
https://codereview.chromium.org/2017053003/ )

Reason for revert:
Will unfortunately have to back out this one so that the revert
https://codereview.chromium.org/2025503002/ will cleanly apply.

(Don't want a tree with this many ASan failures over the long weekend.)

Original issue's description:
> Remove StringBuilder::appendLiteral.
> 
> We can just rely on the append(StringView) version instead. This does
> mean we have to take 2 branches in some cases, but it should be
> very minimal and since the code is inline the compiler can also
> likely eliminate it.
> 
> BUG=615174
> 
> Committed: https://crrev.com/04157e8b89881d033f9eeca4466d9dd0c4e9aaea
> Cr-Commit-Position: refs/heads/master@{#396601}

TBR=haraken@chromium.org,yutak@chromium.org,esprehn@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=615174
==========

to

==========
Revert of Remove StringBuilder::appendLiteral. (patchset #1 id:1 of
https://codereview.chromium.org/2017053003/ )

Reason for revert:
Will unfortunately have to back out this one so that the revert
https://codereview.chromium.org/2025503002/ will cleanly apply.

(Don't want a tree with this many ASan failures over the long weekend.)

Original issue's description:
> Remove StringBuilder::appendLiteral.
>
> We can just rely on the append(StringView) version instead. This does
> mean we have to take 2 branches in some cases, but it should be
> very minimal and since the code is inline the compiler can also
> likely eliminate it.
>
> BUG=615174
>
> Committed: https://crrev.com/04157e8b89881d033f9eeca4466d9dd0c4e9aaea
> Cr-Commit-Position: refs/heads/master@{#396601}

TBR=haraken@chromium.org,yutak@chromium.org,esprehn@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=615174

Committed: https://crrev.com/433566e878a7ce453ae7be38e0d252f8cf804411
Cr-Commit-Position: refs/heads/master@{#396640}
==========

[commit-bot: I haz the power, 2016-05-28 11:51:28]

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