[Sync] Add histogram for events in ClearServerData flow.

[Sync] Add histogram for events in ClearServerData flow.

I'm adding histogram for different events during ClearServerData flow. They'll
help monitor how often enabpling passphrase encryption encounters issues.

R=zea@chromium.org,jwd@chromium.org
BUG=490836
Committed: https://crrev.com/e3f89a3d3785c27d47b884f0c9f0a463adc0e812
Cr-Commit-Position: refs/heads/master@{#437678}

[pavely, 2016-12-07 00:33:01]

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

[commit-bot: I haz the power, 2016-12-07 00:33:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pavely, 2016-12-07 00:35:17]

Description was changed from

==========
[Sync] Add histogram for events in ClearServerData flow.

I'm adding histogram for different events during ClearServerData flow. They'll
help monitor how often enabpling passphrase encryption encounters issues.

R=zea@chromium.org,nparker@chromium.org
BUG=490836
==========

to

==========
[Sync] Add histogram for events in ClearServerData flow.

I'm adding histogram for different events during ClearServerData flow. They'll
help monitor how often enabpling passphrase encryption encounters issues.

R=zea@chromium.org,jwd@chromium.org
BUG=490836
==========

[pavely, 2016-12-07 00:35:17]

pavely@chromium.org changed reviewers:
+ jwd@chromium.org
- nparker@chromium.org

[commit-bot: I haz the power, 2016-12-07 02:15:57]

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

[commit-bot: I haz the power, 2016-12-07 02:15:58]

Dry run: This issue passed the CQ dry run.

[Nicolas Zea, 2016-12-07 21:00:21]

lgtm

https://codereview.chromium.org/2557733004/diff/1/components/browser_sync/pro...
File components/browser_sync/profile_sync_service.cc (right):

https://codereview.chromium.org/2557733004/diff/1/components/browser_sync/pro...
components/browser_sync/profile_sync_service.cc:122: // Events in
ClearServerData flow to be recorded in histogram.
nit: Add a comment that this should not be reordered as it would break
histograms.

[pavely, 2016-12-07 22:05:28]

https://codereview.chromium.org/2557733004/diff/1/components/browser_sync/pro...
File components/browser_sync/profile_sync_service.cc (right):

https://codereview.chromium.org/2557733004/diff/1/components/browser_sync/pro...
components/browser_sync/profile_sync_service.cc:122: // Events in
ClearServerData flow to be recorded in histogram.
On 2016/12/07 21:00:21, Nicolas Zea wrote:
> nit: Add a comment that this should not be reordered as it would break
> histograms.

Done.

[jwd, 2016-12-09 20:04:02]

lgtm

[pavely, 2016-12-09 20:17:38]

The CQ bit was checked by pavely@chromium.org

[pavely, 2016-12-09 20:17:38]

The patchset sent to the CQ was uploaded after l-g-t-m from zea@chromium.org
Link to the patchset: https://codereview.chromium.org/2557733004/#ps20001
(title: "Added comment.")

[commit-bot: I haz the power, 2016-12-09 20:18:59]

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, 2016-12-09 23:00:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481314658194270,
"parent_rev": "70e01e4c1baba55700ac465a9a5cd3e1cb7a01df", "commit_rev":
"4767524c56131adf30ed0a5baddf1b434ab69cea"}

[commit-bot: I haz the power, 2016-12-09 23:00:43]

Description was changed from

==========
[Sync] Add histogram for events in ClearServerData flow.

I'm adding histogram for different events during ClearServerData flow. They'll
help monitor how often enabpling passphrase encryption encounters issues.

R=zea@chromium.org,jwd@chromium.org
BUG=490836
==========

to

==========
[Sync] Add histogram for events in ClearServerData flow.

I'm adding histogram for different events during ClearServerData flow. They'll
help monitor how often enabpling passphrase encryption encounters issues.

R=zea@chromium.org,jwd@chromium.org
BUG=490836

Review-Url: https://codereview.chromium.org/2557733004
==========

[commit-bot: I haz the power, 2016-12-09 23:00:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-12 14:57:13]

Description was changed from

==========
[Sync] Add histogram for events in ClearServerData flow.

I'm adding histogram for different events during ClearServerData flow. They'll
help monitor how often enabpling passphrase encryption encounters issues.

R=zea@chromium.org,jwd@chromium.org
BUG=490836

Review-Url: https://codereview.chromium.org/2557733004
==========

to

==========
[Sync] Add histogram for events in ClearServerData flow.

I'm adding histogram for different events during ClearServerData flow. They'll
help monitor how often enabpling passphrase encryption encounters issues.

R=zea@chromium.org,jwd@chromium.org
BUG=490836

Committed: https://crrev.com/e3f89a3d3785c27d47b884f0c9f0a463adc0e812
Cr-Commit-Position: refs/heads/master@{#437678}
==========

[commit-bot: I haz the power, 2016-12-12 14:57:14]

Patchset 2 (id:??) landed as
https://crrev.com/e3f89a3d3785c27d47b884f0c9f0a463adc0e812
Cr-Commit-Position: refs/heads/master@{#437678}