Revert of Avoid calling strlen() where possible in base/trace_config

Revert of Avoid calling strlen() where possible in base/trace_config (patchset #2 id:20001 of https://codereview.chromium.org/2777203004/ )

Reason for revert:
Broke WebKit Win x64 Builder (dbg) 
blink_platform_unittests:

WorkQueueTest.BlockedByFenceNewFenceUnblocks
WorkQueueTest.TakeTaskFromWorkQueue
WorkQueueTest.InsertFenceAfterEnqueueingNonBlocking
WorkQueueTest.RemoveFence
WorkQueueTest.TakeTaskFromWorkQueue_HitFence
WorkQueueTest.InsertNewFence
WorkQueueTest.BlockedByFencePop
WorkQueueTest.BlockedByFencePopBecomesEmpty

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.webkit%2FWebKit_Win_x64_Builder__dbg_%2F106650%2F%2B%2Frecipes%2Fsteps%2Fblink_platform_unittests%2F0%2Fstdout

Original issue's description:
> 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
> 
> Review-Url: https://codereview.chromium.org/2777203004
> Cr-Commit-Position: refs/heads/master@{#460203}
> Committed: https://chromium.googlesource.com/chromium/src/+/b5d44bb49fd4dc5e544f4312de6b00b1a1eb625c

TBR=primiano@chromium.org,yusukes@chromium.org,iceman@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679479
Review-Url: https://codereview.chromium.org/2776393004
Cr-Commit-Position: refs/heads/master@{#460223}
Committed: https://chromium.googlesource.com/chromium/src/+/dbec06a1bfb0204b72ebd11fafd0885792197043

[imcheng, 2017-03-28 22:13:46]

The CQ bit was checked by imcheng@chromium.org

[imcheng, 2017-03-28 22:13:46]

Created Revert of Avoid calling strlen() where possible in base/trace_config

[commit-bot: I haz the power, 2017-03-28 22:14:15]

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-28 22:15:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490739226061700, "parent_rev":
"f580e12f9cba0b796fad37ae647950940ff2edd5", "commit_rev":
"dbec06a1bfb0204b72ebd11fafd0885792197043"}

[commit-bot: I haz the power, 2017-03-28 22:16:45]

Description was changed from

==========
Revert of Avoid calling strlen() where possible in base/trace_config (patchset
#2 id:20001 of https://codereview.chromium.org/2777203004/ )

Reason for revert:
Broke WebKit Win x64 Builder (dbg) 
blink_platform_unittests:

WorkQueueTest.BlockedByFenceNewFenceUnblocks
WorkQueueTest.TakeTaskFromWorkQueue
WorkQueueTest.InsertFenceAfterEnqueueingNonBlocking
WorkQueueTest.RemoveFence
WorkQueueTest.TakeTaskFromWorkQueue_HitFence
WorkQueueTest.InsertNewFence
WorkQueueTest.BlockedByFencePop
WorkQueueTest.BlockedByFencePopBecomesEmpty

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.webkit%2FWebKit...

Original issue's description:
> 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
> 
> Review-Url: https://codereview.chromium.org/2777203004
> Cr-Commit-Position: refs/heads/master@{#460203}
> Committed:
https://chromium.googlesource.com/chromium/src/+/b5d44bb49fd4dc5e544f4312de6b...

TBR=primiano@chromium.org,yusukes@chromium.org,iceman@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679479
==========

to

==========
Revert of Avoid calling strlen() where possible in base/trace_config (patchset
#2 id:20001 of https://codereview.chromium.org/2777203004/ )

Reason for revert:
Broke WebKit Win x64 Builder (dbg) 
blink_platform_unittests:

WorkQueueTest.BlockedByFenceNewFenceUnblocks
WorkQueueTest.TakeTaskFromWorkQueue
WorkQueueTest.InsertFenceAfterEnqueueingNonBlocking
WorkQueueTest.RemoveFence
WorkQueueTest.TakeTaskFromWorkQueue_HitFence
WorkQueueTest.InsertNewFence
WorkQueueTest.BlockedByFencePop
WorkQueueTest.BlockedByFencePopBecomesEmpty

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.webkit%2FWebKit...

Original issue's description:
> 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
> 
> Review-Url: https://codereview.chromium.org/2777203004
> Cr-Commit-Position: refs/heads/master@{#460203}
> Committed:
https://chromium.googlesource.com/chromium/src/+/b5d44bb49fd4dc5e544f4312de6b...

TBR=primiano@chromium.org,yusukes@chromium.org,iceman@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679479

Review-Url: https://codereview.chromium.org/2776393004
Cr-Commit-Position: refs/heads/master@{#460223}
Committed:
https://chromium.googlesource.com/chromium/src/+/dbec06a1bfb0204b72ebd11fafd0...
==========

[commit-bot: I haz the power, 2017-03-28 22:16:46]

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