Remove std::string usage in PublicMethods.cpp.

Remove std::string usage in PublicMethods.cpp.

BUG=pdfium:488
Committed: https://pdfium.googlesource.com/pdfium/+/cf03f8ee46e66e244bbe57d34c8667e5edc455b3

[Lei Zhang, 2016-05-03 17:53:55]

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

[Tom Sepez, 2016-05-03 18:23:13]

lgtm

[Tom Sepez, 2016-05-03 23:34:30]

https://codereview.chromium.org/1936383002/diff/20001/fpdfsdk/javascript/Publ...
File fpdfsdk/javascript/PublicMethods.cpp (right):

https://codereview.chromium.org/1936383002/diff/20001/fpdfsdk/javascript/Publ...
fpdfsdk/javascript/PublicMethods.cpp:903: CFX_WideString w_strValue = val;
nit: can we lose the _ in these local names?

[Lei Zhang, 2016-05-09 19:22:35]

https://codereview.chromium.org/1936383002/diff/20001/fpdfsdk/javascript/Publ...
File fpdfsdk/javascript/PublicMethods.cpp (right):

https://codereview.chromium.org/1936383002/diff/20001/fpdfsdk/javascript/Publ...
fpdfsdk/javascript/PublicMethods.cpp:903: CFX_WideString w_strValue = val;
On 2016/05/03 23:34:30, Tom Sepez (ooo) wrote:
> nit: can we lose the _ in these local names?

Done.

[Lei Zhang, 2016-05-09 19:22:41]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2016-05-09 19:22:42]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/1936383002/#ps60001
(title: "nits")

[commit-bot: I haz the power, 2016-05-09 19:22:47]

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

[commit-bot: I haz the power, 2016-05-09 19:36:24]

Description was changed from

==========
Remove std::string usage in PublicMethods.cpp.

BUG=pdfium:488
==========

to

==========
Remove std::string usage in PublicMethods.cpp.

BUG=pdfium:488

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

[commit-bot: I haz the power, 2016-05-09 19:36:25]

Committed patchset #4 (id:60001) as
https://pdfium.googlesource.com/pdfium/+/cf03f8ee46e66e244bbe57d34c8667e5edc4...