From 3bf9b147f261b08abfa6ca59064e35b434201202 Mon Sep 17 00:00:00 2001 From: James Hawkins Date: Thu, 3 Mar 2016 14:50:24 -0800 Subject: [PATCH] bootstat: Don't log time_since_factory_reset on device reset. The zero value is meaningless and skews the histogram buckets. Bug: 27477664 Change-Id: Idd641aa3005819905a2ecf664d2a5781dbbff59b --- bootstat/bootstat.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/bootstat/bootstat.cpp b/bootstat/bootstat.cpp index c199190eb..bcc8abdf5 100644 --- a/bootstat/bootstat.cpp +++ b/bootstat/bootstat.cpp @@ -170,7 +170,9 @@ void RecordFactoryReset() { // use this signal to mark the time of the factory reset. if (!boot_event_store.GetBootEvent("factory_reset", &record)) { boot_event_store.AddBootEventWithValue("factory_reset", current_time_utc); - boot_event_store.AddBootEventWithValue("time_since_factory_reset", 0); + + // Don't log the time_since_factory_reset until some time has elapsed. + // The data is not meaningful yet and skews the histogram buckets. return; }