Avoid needless construction of CFX_ByteStrings during string building

Avoid needless construction of CFX_ByteStrings during string building

There are perfectly fine operators for adding to an existing string
without first duplicating the arguments.

Committed: https://pdfium.googlesource.com/pdfium/+/ff242e0b62173718a6e98aee1306ac48f6372572

[Tom Sepez, 2016-04-14 21:21:30]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[Tom Sepez, 2016-04-14 21:21:31]

Lei, for review.

[Lei Zhang, 2016-04-14 21:34:50]

lgtm

[Tom Sepez, 2016-04-14 21:38:51]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-04-14 21:39:01]

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

[commit-bot: I haz the power, 2016-04-14 21:55:01]

Description was changed from

==========
Avoid needless construction of CFX_ByteStrings during string building

There are perfectly fine operators for adding to an existing string
without first duplicating the arguments.
==========

to

==========
Avoid needless construction of CFX_ByteStrings during string building

There are perfectly fine operators for adding to an existing string
without first duplicating the arguments.

Committed:
https://pdfium.googlesource.com/pdfium/+/ff242e0b62173718a6e98aee1306ac48f637...
==========

[commit-bot: I haz the power, 2016-04-14 21:55:02]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/ff242e0b62173718a6e98aee1306ac48f637...