Fix test launcher crash on non-UTF-8 output snippets

Fix test launcher crash on non-UTF-8 output snippets

BUG=525484
Committed: https://crrev.com/cd6ee1a8479878323e7a3e54b6588038428876b2
Cr-Commit-Position: refs/heads/master@{#346098}

[Paweł Hajdan Jr., 2015-08-27 14:49:23]

phajdan.jr@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2015-08-27 16:51:24]

LGTM

[sky, 2015-08-27 16:51:56]

https://codereview.chromium.org/1312923003/diff/1/base/test/launcher/test_res...
File base/test/launcher/test_results_tracker.cc (right):

https://codereview.chromium.org/1312923003/diff/1/base/test/launcher/test_res...
base/test/launcher/test_results_tracker.cc:273: "<non-UTF-8 snippet, see
output_snippet_base64>");
Actually, one question here, can we escape the string so that you see at least
something?

[Paweł Hajdan Jr., 2015-08-28 08:04:24]

https://codereview.chromium.org/1312923003/diff/1/base/test/launcher/test_res...
File base/test/launcher/test_results_tracker.cc (right):

https://codereview.chromium.org/1312923003/diff/1/base/test/launcher/test_res...
base/test/launcher/test_results_tracker.cc:273: "<non-UTF-8 snippet, see
output_snippet_base64>");
On 2015/08/27 at 16:51:56, sky wrote:
> Actually, one question here, can we escape the string so that you see at least
something?

That's what the previous code was trying to do. Apparently it doesn't work. :-/

Since this is a serious issue affecting stability and CQ flakiness, I applied a
solution I'm pretty sure will work.

We can think about next steps after that.

Note that this is the snippet in JSON summary, not what is printed to stdout
(although I may need to change the latter as well).

[Paweł Hajdan Jr., 2015-08-28 08:04:26]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2015-08-28 08:04:34]

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

[commit-bot: I haz the power, 2015-08-28 08:09:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-08-28 08:09:46]

Patchset 1 (id:??) landed as
https://crrev.com/cd6ee1a8479878323e7a3e54b6588038428876b2
Cr-Commit-Position: refs/heads/master@{#346098}