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

Revert of [Android] Stop using universal_newlines in the layout tests server constructor. (patchset #2 id:20001 of https://codereview.chromium.org/2629113005/ )

Reason for revert:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/59416

Original issue's description:
> [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

TBR=qyearsley@chromium.org,jeffcarp@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679723, 496983
Review-Url: https://codereview.chromium.org/2631743003
Cr-Commit-Position: refs/heads/master@{#443693}
Committed: https://chromium.googlesource.com/chromium/src/+/5489020f1edea26f4256621b874afebf7425fdf0

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

The CQ bit was checked by jbudorick@chromium.org

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

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

[commit-bot: I haz the power, 2017-01-13 22:09:28]

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 22:10:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484345324750190, "parent_rev":
"af8f03fcb37a1bb4bf1935c060880d82297350e7", "commit_rev":
"5489020f1edea26f4256621b874afebf7425fdf0"}

[commit-bot: I haz the power, 2017-01-13 22:10:43]

Description was changed from

==========
Revert of [Android] Stop using universal_newlines in the layout tests server
constructor. (patchset #2 id:20001 of
https://codereview.chromium.org/2629113005/ )

Reason for revert:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28N...

Original issue's description:
> [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...

TBR=qyearsley@chromium.org,jeffcarp@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679723,496983
==========

to

==========
Revert of [Android] Stop using universal_newlines in the layout tests server
constructor. (patchset #2 id:20001 of
https://codereview.chromium.org/2629113005/ )

Reason for revert:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28N...

Original issue's description:
> [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...

TBR=qyearsley@chromium.org,jeffcarp@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679723,496983

Review-Url: https://codereview.chromium.org/2631743003
Cr-Commit-Position: refs/heads/master@{#443693}
Committed:
https://chromium.googlesource.com/chromium/src/+/5489020f1edea26f4256621b874a...
==========

[commit-bot: I haz the power, 2017-01-13 22:10:44]

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

[qyearsley, 2017-04-20 20:54:30]

lgtm

(Going through old CLs that I hadn't responded to...)