From fba665e82bec87a511b5cc20beaaf75bb20276e9 Mon Sep 17 00:00:00 2001 From: silviavinyes Date: Fri, 29 Jul 2022 13:46:07 +0100 Subject: [PATCH] Add logging for ambient light. Bug: 243369305 Test: statsd_testdrive Change-Id: I7b3a823f116b68a349d398b9cd2dab465e27ea9c (cherry picked from commit 642112e475b040b874ade8f43328b7faad36061b) Merged-in: I7b3a823f116b68a349d398b9cd2dab465e27ea9c --- .../server/display/AmbientBrightnessStatsTracker.java | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/services/core/java/com/android/server/display/AmbientBrightnessStatsTracker.java b/services/core/java/com/android/server/display/AmbientBrightnessStatsTracker.java index d74c7025ae333..7c9a48431a249 100644 --- a/services/core/java/com/android/server/display/AmbientBrightnessStatsTracker.java +++ b/services/core/java/com/android/server/display/AmbientBrightnessStatsTracker.java @@ -27,6 +27,7 @@ import android.util.TypedXmlSerializer; import android.util.Xml; import com.android.internal.annotations.VisibleForTesting; +import com.android.internal.util.FrameworkStatsLog; import org.xmlpull.v1.XmlPullParser; import org.xmlpull.v1.XmlPullParserException; @@ -287,6 +288,14 @@ public class AmbientBrightnessStatsTracker { localDate)) { return lastBrightnessStats; } else { + // It is a new day, and we have available data, so log data. The daily boundary + // might not be right if the user changes timezones but that is fine, since it + // won't be that frequent. + if (lastBrightnessStats != null) { + FrameworkStatsLog.write(FrameworkStatsLog.AMBIENT_BRIGHTNESS_STATS_REPORTED, + lastBrightnessStats.getStats(), + lastBrightnessStats.getBucketBoundaries()); + } AmbientBrightnessDayStats dayStats = new AmbientBrightnessDayStats(localDate, BUCKET_BOUNDARIES_FOR_NEW_STATS); if (userStats.size() == MAX_DAYS_TO_TRACK) {