Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode exception

Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode exception

Reason for reland:
Had a DCHECK that was removing an important expression.
Ensured that net_unittests failed locally, and was fixed with change.

No behaviour change expected. Strictly a simplifying refactoring.

Review-Url: https://codereview.chromium.org/2942133002
Cr-Commit-Position: refs/heads/master@{#480057}
Committed: https://chromium.googlesource.com/chromium/src/+/8e74adc3a8fd143e8624deeeb07ee73750553dd2

[agrieve, 2017-06-16 01:42:48]

Created Reland of Android: Refactor BuildInfo to use less jni and remove
StrictMode exception

[agrieve, 2017-06-16 02:43:49]

Description was changed from

==========
Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode
exception (patchset #1 id:1 of https://codereview.chromium.org/2835113004/ )

Reason for revert:
Will update when ready

Original issue's description:
> Revert of Android: Refactor BuildInfo to use less jni and remove StrictMode
exception (patchset #2 id:20001 of https://codereview.chromium.org/2825233002/ )
> 
> Reason for revert:
> Likely causing net_unittest failures.
> 
> e.g.:
>
https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20Mar...
> 
> Original issue's description:
> > Android: Refactor BuildInfo to use less jni and remove StrictMode exception
> > 
> > No behaviour change expected. Strictly a simplifying refactoring.
> > 
> > Review-Url: https://codereview.chromium.org/2825233002
> > Cr-Commit-Position: refs/heads/master@{#466630}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/c4cb0d38f0bd26578120775e9c09...
> 
> TBR=torne@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> 
> Review-Url: https://codereview.chromium.org/2835113004
> Cr-Commit-Position: refs/heads/master@{#466633}
> Committed:
https://chromium.googlesource.com/chromium/src/+/91522d0519823a43060738914a65...

TBR=torne@chromium.org,mgersh@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode
exception

Reason for reland:
Had a DCHECK that was removing an important expression.
Ensured that net_unittests failed locally, and was fixed with change.

No behaviour change expected. Strictly a simplifying refactoring.

TBR=torne@chromium.org,mgersh@chromium.org
==========

[agrieve, 2017-06-16 02:44:54]

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

[commit-bot: I haz the power, 2017-06-16 02:45:11]

Dry run: 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-06-16 03:50:07]

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

[commit-bot: I haz the power, 2017-06-16 03:50:07]

Dry run: This issue passed the CQ dry run.

[Torne, 2017-06-16 14:53:01]

lgtm

[agrieve, 2017-06-16 15:53:54]

Description was changed from

==========
Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode
exception

Reason for reland:
Had a DCHECK that was removing an important expression.
Ensured that net_unittests failed locally, and was fixed with change.

No behaviour change expected. Strictly a simplifying refactoring.

TBR=torne@chromium.org,mgersh@chromium.org
==========

to

==========
Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode
exception

Reason for reland:
Had a DCHECK that was removing an important expression.
Ensured that net_unittests failed locally, and was fixed with change.

No behaviour change expected. Strictly a simplifying refactoring.
==========

[agrieve, 2017-06-16 15:54:04]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2017-06-16 15:54:27]

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-06-16 15:57:29]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1497628444188180,
"parent_rev": "1d3617187c31c8c00241f479e4445c21971d3d80", "commit_rev":
"8e74adc3a8fd143e8624deeeb07ee73750553dd2"}

[commit-bot: I haz the power, 2017-06-16 15:57:41]

Description was changed from

==========
Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode
exception

Reason for reland:
Had a DCHECK that was removing an important expression.
Ensured that net_unittests failed locally, and was fixed with change.

No behaviour change expected. Strictly a simplifying refactoring.
==========

to

==========
Reland of Android: Refactor BuildInfo to use less jni and remove StrictMode
exception

Reason for reland:
Had a DCHECK that was removing an important expression.
Ensured that net_unittests failed locally, and was fixed with change.

No behaviour change expected. Strictly a simplifying refactoring.

Review-Url: https://codereview.chromium.org/2942133002
Cr-Commit-Position: refs/heads/master@{#480057}
Committed:
https://chromium.googlesource.com/chromium/src/+/8e74adc3a8fd143e8624deeeb07e...
==========

[commit-bot: I haz the power, 2017-06-16 15:57:43]

Committed patchset #2 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/8e74adc3a8fd143e8624deeeb07e...