Reland: Avoid calling strlen() where possible in base/trace_config

Reland: Avoid calling strlen() where possible in base/trace_config

Fix for crbug.com/706416 was landed and this patch should work fine.

The main target of this CL is to avoid calling strlen() where possible.
It can happen if a raw |char*| literal or result of |c_str()| function is used
to construct a StringPiece or std::string objects.

In addition, StringPiece API like SplitStringPiece was used to avoid
unnecessary string copying and allocations.

BUG=679479
TBR=primiano@chromium.org,yusukes@chromium.org

Review-Url: https://codereview.chromium.org/2780983003
Cr-Commit-Position: refs/heads/master@{#460720}
Committed: https://chromium.googlesource.com/chromium/src/+/5e8d28b598fa533bd3c0503e59e037d63913fbde

[Andrey Kraynov, 2017-03-29 21:30:00]

Hi!

This is a reland of my CL https://codereview.chromium.org/2777203004/
It was landed and then reverted due to broken Blink tests (crbug.com/706416).
Now tests code was fixed and I can try one more time.
I've rebased and tested it on latest master.

PTAL.

[Yusuke Sato, 2017-03-30 00:53:46]

arc/ lgtm

[Primiano Tucci (use gerrit), 2017-03-30 09:13:09]

lgtm

[Primiano Tucci (use gerrit), 2017-03-30 09:13:13]

The CQ bit was checked by primiano@chromium.org

[commit-bot: I haz the power, 2017-03-30 09:13:25]

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, 2017-03-30 11:22:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490865193469070, "parent_rev":
"38f2a30295a7b589b5746c3c7d1a6a5ef6cc3438", "commit_rev":
"5e8d28b598fa533bd3c0503e59e037d63913fbde"}

[commit-bot: I haz the power, 2017-03-30 11:22:57]

Description was changed from

==========
Reland: Avoid calling strlen() where possible in base/trace_config

Fix for crbug.com/706416 was landed and this patch should work fine.

The main target of this CL is to avoid calling strlen() where possible.
It can happen if a raw |char*| literal or result of |c_str()| function is used
to construct a StringPiece or std::string objects.

In addition, StringPiece API like SplitStringPiece was used to avoid
unnecessary string copying and allocations.

BUG=679479
TBR=primiano@chromium.org,yusukes@chromium.org
==========

to

==========
Reland: Avoid calling strlen() where possible in base/trace_config

Fix for crbug.com/706416 was landed and this patch should work fine.

The main target of this CL is to avoid calling strlen() where possible.
It can happen if a raw |char*| literal or result of |c_str()| function is used
to construct a StringPiece or std::string objects.

In addition, StringPiece API like SplitStringPiece was used to avoid
unnecessary string copying and allocations.

BUG=679479
TBR=primiano@chromium.org,yusukes@chromium.org

Review-Url: https://codereview.chromium.org/2780983003
Cr-Commit-Position: refs/heads/master@{#460720}
Committed:
https://chromium.googlesource.com/chromium/src/+/5e8d28b598fa533bd3c0503e59e0...
==========

[commit-bot: I haz the power, 2017-03-30 11:22:58]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5e8d28b598fa533bd3c0503e59e0...