Avoid double initializing reporting_service_

Avoid double initializing reporting_service_

Issue introduced in https://codereview.chromium.org/2608833002

ReportingService is already initialized by InitializeMetricsRecordingState, don't call Initialize again in InitializeMetricsState.

Adds a DCHECK to Initialize to make sure it isn't called twice.

BUG=693676
Review-Url: https://codereview.chromium.org/2752073002
Cr-Commit-Position: refs/heads/master@{#457214}
Committed: https://chromium.googlesource.com/chromium/src/+/5576ba8067a8c06eec493384d17acec923b0d2b3

[Steven Holte, 2017-03-15 19:39:16]

holte@chromium.org changed reviewers:
+ asvitkine@chromium.org, penghuang@chromium.org

[Steven Holte, 2017-03-15 19:39:55]

Description was changed from

==========
Avoid double initializing reporting_service_

ReportingService is already initialized by InitializeMetricsRecordingState,
don't call Initialize again in InitializeMetricsState.

Adds a DCHECK to Initialize to make sure it isn't called twice.

BUG=693676
==========

to

==========
Avoid double initializing reporting_service_

Issue introduced in https://codereview.chromium.org/2608833002

ReportingService is already initialized by InitializeMetricsRecordingState,
don't call Initialize again in InitializeMetricsState.

Adds a DCHECK to Initialize to make sure it isn't called twice.

BUG=693676
==========

[Alexei Svitkine (slow), 2017-03-15 20:00:15]

lgtm

[Steven Holte, 2017-03-15 20:22:37]

The CQ bit was checked by holte@chromium.org

[commit-bot: I haz the power, 2017-03-15 20:23:06]

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-03-15 21:26:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489609357482090, "parent_rev":
"fd3496f6cad18623fde35367afc4c57080e0044e", "commit_rev":
"5576ba8067a8c06eec493384d17acec923b0d2b3"}

[commit-bot: I haz the power, 2017-03-15 21:28:16]

Description was changed from

==========
Avoid double initializing reporting_service_

Issue introduced in https://codereview.chromium.org/2608833002

ReportingService is already initialized by InitializeMetricsRecordingState,
don't call Initialize again in InitializeMetricsState.

Adds a DCHECK to Initialize to make sure it isn't called twice.

BUG=693676
==========

to

==========
Avoid double initializing reporting_service_

Issue introduced in https://codereview.chromium.org/2608833002

ReportingService is already initialized by InitializeMetricsRecordingState,
don't call Initialize again in InitializeMetricsState.

Adds a DCHECK to Initialize to make sure it isn't called twice.

BUG=693676

Review-Url: https://codereview.chromium.org/2752073002
Cr-Commit-Position: refs/heads/master@{#457214}
Committed:
https://chromium.googlesource.com/chromium/src/+/5576ba8067a8c06eec493384d17a...
==========

[commit-bot: I haz the power, 2017-03-15 21:28:17]

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