Generalized the storage strategy used by UpdateProfileCounts.

Generalized the storage strategy used by UpdateProfileCounts.

This simplifies adding new such stats.

BUG=643647
TEST=covered by GoogleUpdateSettingsTest.UpdateProfileCounts*

Committed: https://crrev.com/f7c30a5abe40796ca0972f541324c6a117d3ca64
Cr-Commit-Position: refs/heads/master@{#416724}

[grt (UTC plus 2), 2016-09-05 07:57:31]

Patchset #1 (id:1) has been deleted

[grt (UTC plus 2), 2016-09-05 07:58:53]

Description was changed from

==========
Generalized the storage strategy used by UpdateProfileCounts.

This simplifies adding new such stats.

BUG=643647
==========

to

==========
Generalized the storage strategy used by UpdateProfileCounts.

This simplifies adding new such stats.

BUG=643647
TEST=covered by GoogleUpdateSettingsTest.UpdateProfileCounts*
==========

[grt (UTC plus 2), 2016-09-05 07:58:53]

grt@chromium.org changed reviewers:
+ bcwhite@chromium.org

[grt (UTC plus 2), 2016-09-05 07:59:31]

Hi Brian. Time for a quick review?

[grt (UTC plus 2), 2016-09-05 07:59:37]

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

[commit-bot: I haz the power, 2016-09-05 07:59:46]

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, 2016-09-05 08:48:14]

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

[commit-bot: I haz the power, 2016-09-05 08:48:15]

Dry run: This issue passed the CQ dry run.

[bcwhite, 2016-09-06 17:07:39]

lgtm

[grt (UTC plus 2), 2016-09-06 19:55:28]

The CQ bit was checked by grt@chromium.org

[commit-bot: I haz the power, 2016-09-06 19:56:13]

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-09-06 20:52:49]

Description was changed from

==========
Generalized the storage strategy used by UpdateProfileCounts.

This simplifies adding new such stats.

BUG=643647
TEST=covered by GoogleUpdateSettingsTest.UpdateProfileCounts*
==========

to

==========
Generalized the storage strategy used by UpdateProfileCounts.

This simplifies adding new such stats.

BUG=643647
TEST=covered by GoogleUpdateSettingsTest.UpdateProfileCounts*
==========

[commit-bot: I haz the power, 2016-09-06 20:52:50]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-09-06 20:55:27]

Description was changed from

==========
Generalized the storage strategy used by UpdateProfileCounts.

This simplifies adding new such stats.

BUG=643647
TEST=covered by GoogleUpdateSettingsTest.UpdateProfileCounts*
==========

to

==========
Generalized the storage strategy used by UpdateProfileCounts.

This simplifies adding new such stats.

BUG=643647
TEST=covered by GoogleUpdateSettingsTest.UpdateProfileCounts*

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

[commit-bot: I haz the power, 2016-09-06 20:55:28]

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