Make sure StatisticsRecorder::Initialize() is called at initialization

Make sure StatisticsRecorder::Initialize() is called at initialization

Also, add DCHECK that CronetEnvironment::Initialize() is called on the
main thread.

The current implementation guarantees that when [Cronet start] method
returns, the CronetEnvironment singleton is initialized, and, as the
result, base::StatisticsRecorder::Initialize() has been invoked.

BUG=707836
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2885543002
Cr-Commit-Position: refs/heads/master@{#472555}
Committed: https://chromium.googlesource.com/chromium/src/+/a3c5a020f874bbd3a311ab325b541c4db2e100ac

[kapishnikov, 2017-05-15 19:27:18]

Description was changed from

==========
Make sure StatisticsRecorder::Initialize() is called at initialization

Also, add DCHECK that CronetEnvironment::Initialize() is called on the
main thread.

The current implementation guarantees that when [Cronet start] method
returns, the CronetEnvironment singleton is initialized, and, as the
result, base::StatisticsRecorder::Initialize() has been invoked.

BUG=707836
==========

to

==========
Make sure StatisticsRecorder::Initialize() is called at initialization

Also, add DCHECK that CronetEnvironment::Initialize() is called on the
main thread.

The current implementation guarantees that when [Cronet start] method
returns, the CronetEnvironment singleton is initialized, and, as the
result, base::StatisticsRecorder::Initialize() has been invoked.

BUG=707836
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[kapishnikov, 2017-05-15 19:28:40]

kapishnikov@chromium.org changed reviewers:
+ lilyhoughton@chromium.org, mef@chromium.org

[kapishnikov, 2017-05-15 19:28:40]

PTAL

[mef, 2017-05-17 20:11:43]

lgtm

[kapishnikov, 2017-05-17 20:12:08]

The CQ bit was checked by kapishnikov@chromium.org

[commit-bot: I haz the power, 2017-05-17 20:13:34]

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-05-17 20:51:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495051928119470, "parent_rev":
"d77878b9c9f4f4db96a21e0974843a7e8cd77b24", "commit_rev":
"a3c5a020f874bbd3a311ab325b541c4db2e100ac"}

[commit-bot: I haz the power, 2017-05-17 20:51:42]

Description was changed from

==========
Make sure StatisticsRecorder::Initialize() is called at initialization

Also, add DCHECK that CronetEnvironment::Initialize() is called on the
main thread.

The current implementation guarantees that when [Cronet start] method
returns, the CronetEnvironment singleton is initialized, and, as the
result, base::StatisticsRecorder::Initialize() has been invoked.

BUG=707836
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Make sure StatisticsRecorder::Initialize() is called at initialization

Also, add DCHECK that CronetEnvironment::Initialize() is called on the
main thread.

The current implementation guarantees that when [Cronet start] method
returns, the CronetEnvironment singleton is initialized, and, as the
result, base::StatisticsRecorder::Initialize() has been invoked.

BUG=707836
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

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

[commit-bot: I haz the power, 2017-05-17 20:51:43]

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