[Variations] Enable Safe Mode in monitoring mode

Desktop / Chromium - Ilya Sherman [chromium.org] - 14 July 2017 22:17 EDT

Prior to launching Safe Mode, we want to verify that there won't be a significant number of false positives, which might otherwise cause more harm than good. This CL adds metrics to measure how often Safe Mode would trigger, without actually enabling safe mode even when the heuristics say it should trigger.

R=asvitkine@chromium.org

Bug: 727984 Change-Id: I83b909bf81ec8c6416f47151c17fe4e53725ee3c Reviewed-on: https://chromium-review.googlesource.com/570731 Commit-Queue: Ilya Sherman

f7a44e1 [Variations] Enable Safe Mode in monitoring mode.
components/variations/pref_names.cc | 9 +
components/variations/pref_names.h | 2 +
.../variations/service/variations_service.cc | 61 +++++-
components/variations/service/variations_service.h | 9 +
.../service/variations_service_unittest.cc | 220 ++++++++++++++++++++-
tools/metrics/histograms/enums.xml | 5 +
tools/metrics/histograms/histograms.xml | 27 +++
7 files changed, 328 insertions(+), 5 deletions(-)

Upstream: git.chromium.org


  • Share