[Android] Stop using universal_newlines in the layout tests server constructor.

[Android] Stop using universal_newlines in the layout tests server constructor.

This had previously been necessary when spawning adb instances with the
server constructor, as adb mangles newlines. Starting with
https://codereview.chromium.org/2540603004/, the android port no longer
spins up adb shells with the server constructor, so we no longer need to
use universal_newlines. This also means we no longer need to encode crash
streams back into bytes from text, as they're not converted to text in the
first place.

BUG=679723, 496983
Review-Url: https://codereview.chromium.org/2629113005
Cr-Commit-Position: refs/heads/master@{#443621}
Committed: https://chromium.googlesource.com/chromium/src/+/f7be987d1b081aa826af6d842e27608dd47a7cff

[jbudorick, 2017-01-13 14:57:34]

Description was changed from

==========
[Android] Stop using universal_newlines in the layout tests server constructor.

This had previously been necessary when spawning adb instances with the
server constructor, as adb mangles newlines. Starting with
https://codereview.chromium.org/2540603004/, the android port no longer
spins up adb shells with the server constructor, so we no longer need to
use universal_newlines. This also means we no longer need to encode crash
streams back into bytes from text, as they're not converted to text in the
first place.

BUG=679723
==========

to

==========
[Android] Stop using universal_newlines in the layout tests server constructor.

This had previously been necessary when spawning adb instances with the
server constructor, as adb mangles newlines. Starting with
https://codereview.chromium.org/2540603004/, the android port no longer
spins up adb shells with the server constructor, so we no longer need to
use universal_newlines. This also means we no longer need to encode crash
streams back into bytes from text, as they're not converted to text in the
first place.

BUG=679723,496983
==========

[jbudorick, 2017-01-13 15:10:38]

jbudorick@chromium.org changed reviewers:
+ jeffcarp@chromium.org, qyearsley@chromium.org

[qyearsley, 2017-01-13 17:20:25]

Excellent, LGTM

[jbudorick, 2017-01-13 17:21:50]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2017-01-13 17:22:26]

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-01-13 18:34:24]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484328110067850,
"parent_rev": "63e22b6fcb629ebccc0f257de04bc8e7f4ef247c", "commit_rev":
"f7be987d1b081aa826af6d842e27608dd47a7cff"}

[commit-bot: I haz the power, 2017-01-13 18:35:01]

Description was changed from

==========
[Android] Stop using universal_newlines in the layout tests server constructor.

This had previously been necessary when spawning adb instances with the
server constructor, as adb mangles newlines. Starting with
https://codereview.chromium.org/2540603004/, the android port no longer
spins up adb shells with the server constructor, so we no longer need to
use universal_newlines. This also means we no longer need to encode crash
streams back into bytes from text, as they're not converted to text in the
first place.

BUG=679723,496983
==========

to

==========
[Android] Stop using universal_newlines in the layout tests server constructor.

This had previously been necessary when spawning adb instances with the
server constructor, as adb mangles newlines. Starting with
https://codereview.chromium.org/2540603004/, the android port no longer
spins up adb shells with the server constructor, so we no longer need to
use universal_newlines. This also means we no longer need to encode crash
streams back into bytes from text, as they're not converted to text in the
first place.

BUG=679723,496983

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

[commit-bot: I haz the power, 2017-01-13 18:35:02]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/f7be987d1b081aa826af6d842e27...

[jbudorick, 2017-01-13 22:08:44]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2631743003/ by jbudorick@chromium.org.

The reason for reverting is:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28N....