Description[Cleanup] Clean up the VariationsSeedStore's histograms.
* Rename Variations.SeedEmpty to Variations.SeedLoadResult, and only record it
when loading the seed on startup. (Previously, half of the enum values were
also recorded when storing a seed, which makes the metric quite confusing to
interpret.)
* Use enum classes in place of regular enums (IMO this makes the constant names
a bit easier to read.)
* Move most of the metrics recording code out of the variations_seed_store.cc
implementation file into a separate helper file.
BUG=none
TEST=none
R=asvitkine@chromium.org
Review-Url: https://codereview.chromium.org/2935623004
Cr-Commit-Position: refs/heads/master@{#479411}
Committed: https://chromium.googlesource.com/chromium/src/+/5f556b51553f6a8b163987dc56594f337192f38d
Patch Set 1 #Patch Set 2 : Remove a copy/pasta error #Patch Set 3 : Fix the unit tests #Patch Set 4 : Fix tests on iOS and Android #
Total comments: 4
Patch Set 5 : Test signature verification on Android and iOS too. #Patch Set 6 : Rebase #
Messages
Total messages: 31 (25 generated)
|