Remove invalid UTF-8 characters from test output

Remove invalid UTF-8 characters from test output

Occasionally tests output characters that aren't valid unicode UTF-8
characters.  This causes the --json-test-results file not to be written.
Replace these characters with the UTF-8 invalid character.

BUG=

Committed: https://crrev.com/0f75d7d3e3f890d86fbb3d7616fbfa469b41ef76
Cr-Commit-Position: refs/heads/master@{#37404}

[brendan.kirby, 2016-06-23 00:59:55]

brendan.kirby@imgtec.com changed reviewers:
+ jkummerow@chromium.org, machenbach@chromium.org, yangguo@chromium.org

[brendan.kirby, 2016-06-23 00:59:55]

PTAL We're seeing some V8 tests fail that are writing invalid UTF-8 characters
in their output.  run-tests.py fails to generate the JSON test results file with
these characters in it.  This means we don't get the durations file or the test
results in Buildbot. This patch removes those characters.

[Michael Achenbach, 2016-06-24 08:09:57]

The CQ bit was checked by machenbach@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-24 08:10:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2086143003/1

[commit-bot: I haz the power, 2016-06-24 08:34:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-24 08:34:17]

Dry run: This issue passed the CQ dry run.

[Michael Achenbach, 2016-06-24 08:47:37]

lgtm - seems legit. A place where this could go wrong is the checked in webkit
test expectations (we make text comparison). But since the dry-run passed, I
guess there are not problematic characters in there...

[Yang, 2016-06-27 08:00:24]

On 2016/06/24 08:47:37, Michael Achenbach wrote:
> lgtm - seems legit. A place where this could go wrong is the checked in webkit
> test expectations (we make text comparison). But since the dry-run passed, I
> guess there are not problematic characters in there...

lgtm.

[brendan.kirby, 2016-06-29 19:48:06]

The CQ bit was checked by brendan.kirby@imgtec.com

[commit-bot: I haz the power, 2016-06-29 19:48:12]

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, 2016-06-29 20:32:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-29 20:35:01]

Description was changed from

==========
Remove invalid UTF-8 characters from test output

Occasionally tests output characters that aren't valid unicode UTF-8
characters.  This causes the --json-test-results file not to be written.
Replace these characters with the UTF-8 invalid character.

BUG=
==========

to

==========
Remove invalid UTF-8 characters from test output

Occasionally tests output characters that aren't valid unicode UTF-8
characters.  This causes the --json-test-results file not to be written.
Replace these characters with the UTF-8 invalid character.

BUG=

Committed: https://crrev.com/0f75d7d3e3f890d86fbb3d7616fbfa469b41ef76
Cr-Commit-Position: refs/heads/master@{#37404}
==========

[commit-bot: I haz the power, 2016-06-29 20:35:02]

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