Still more comments about strings

Still more comments about strings

TBR=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/80b103bf53ee1249823e6bb3bed9cb2bc15ca67a

[Tom Sepez, 2016-04-15 00:09:38]

Lei, even more accurate comments.

[Tom Sepez, 2016-04-15 00:09:59]

https://codereview.chromium.org/1888843004/diff/1/core/fxcrt/include/fx_string.h
File core/fxcrt/include/fx_string.h (right):

https://codereview.chromium.org/1888843004/diff/1/core/fxcrt/include/fx_strin...
core/fxcrt/include/fx_string.h:367: // |ch| must be an lvalue that outlives the
the CFX_WideStringC. However,
all cut n paste.

[Tom Sepez, 2016-04-15 00:10:02]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-04-15 00:10:07]

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

[Tom Sepez, 2016-04-15 00:19:18]

The CQ bit was unchecked by tsepez@chromium.org

[Tom Sepez, 2016-04-15 00:20:26]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-04-15 00:20:37]

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

[Lei Zhang, 2016-04-15 00:37:40]

lgtm

[commit-bot: I haz the power, 2016-04-15 00:39:13]

Description was changed from

==========
Still more comments about strings

TBR=thestig@chromium.org
==========

to

==========
Still more comments about strings

TBR=thestig@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/80b103bf53ee1249823e6bb3bed9cb2bc15c...
==========

[commit-bot: I haz the power, 2016-04-15 00:39:14]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/80b103bf53ee1249823e6bb3bed9cb2bc15c...