Use the single char overload of append() when possible.

Use the single char overload of append() when possible.

We should use StringBuilder::append(LChar) or String::append(LChar) when
possible since it produces smaller code and is faster. I also switched
some code that was calling String::append() repeatedly to use
StringBuilder instead to avoid the repeated copies.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Committed: https://crrev.com/0ea8294b494133a91a8d6c9dbeae031945143528
Cr-Commit-Position: refs/heads/master@{#404964}

[esprehn, 2016-07-11 22:58:38]

Description was changed from

==========
Use the single char overload of append() when possible.

We should use StringBuilder::append(LChar) or String::append(LChar) when
possible since it produces smaller code and is faster. I also switched
some code that was calling String::append() repeatedly to use
StringBuilder instead to avoid the repeated copies.
==========

to

==========
Use the single char overload of append() when possible.

We should use StringBuilder::append(LChar) or String::append(LChar) when
possible since it produces smaller code and is faster. I also switched
some code that was calling String::append() repeatedly to use
StringBuilder instead to avoid the repeated copies.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[esprehn, 2016-07-11 22:59:33]

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

[esprehn, 2016-07-11 22:59:50]

Description was changed from

==========
Use the single char overload of append() when possible.

We should use StringBuilder::append(LChar) or String::append(LChar) when
possible since it produces smaller code and is faster. I also switched
some code that was calling String::append() repeatedly to use
StringBuilder instead to avoid the repeated copies.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Use the single char overload of append() when possible.

We should use StringBuilder::append(LChar) or String::append(LChar) when
possible since it produces smaller code and is faster. I also switched
some code that was calling String::append() repeatedly to use
StringBuilder instead to avoid the repeated copies.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[commit-bot: I haz the power, 2016-07-11 23:00:24]

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, 2016-07-12 00:29:32]

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

[commit-bot: I haz the power, 2016-07-12 00:29:32]

Dry run: Try jobs failed on following builders:
  win_optional_gpu_tests_rel on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_...)

[esprehn, 2016-07-12 00:40:00]

esprehn@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2016-07-12 00:58:06]

LGTM

[esprehn, 2016-07-12 01:19:35]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-07-12 01:20:32]

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-07-12 03:18:36]

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

[commit-bot: I haz the power, 2016-07-12 03:18:38]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[esprehn, 2016-07-12 03:30:36]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-07-12 03:32:01]

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-07-12 06:43:57]

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

[commit-bot: I haz the power, 2016-07-12 06:43:59]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[esprehn, 2016-07-12 19:09:36]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-07-12 19:12:24]

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-07-12 22:42:39]

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

[commit-bot: I haz the power, 2016-07-12 22:42:41]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[esprehn, 2016-07-12 22:50:06]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-07-12 22:53: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, 2016-07-13 01:08:08]

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

[commit-bot: I haz the power, 2016-07-13 01:08:11]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[esprehn, 2016-07-13 01:51:37]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-07-13 01:54:36]

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-07-13 03:31:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-13 03:32:41]

Description was changed from

==========
Use the single char overload of append() when possible.

We should use StringBuilder::append(LChar) or String::append(LChar) when
possible since it produces smaller code and is faster. I also switched
some code that was calling String::append() repeatedly to use
StringBuilder instead to avoid the repeated copies.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Use the single char overload of append() when possible.

We should use StringBuilder::append(LChar) or String::append(LChar) when
possible since it produces smaller code and is faster. I also switched
some code that was calling String::append() repeatedly to use
StringBuilder instead to avoid the repeated copies.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Committed: https://crrev.com/0ea8294b494133a91a8d6c9dbeae031945143528
Cr-Commit-Position: refs/heads/master@{#404964}
==========

[commit-bot: I haz the power, 2016-07-13 03:32:43]

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