From 2697445dcd01b67f248e3d616188c8169981457e Mon Sep 17 00:00:00 2001 From: Jiaming Liu Date: Tue, 8 Nov 2022 02:05:33 +0000 Subject: [PATCH] Add error logs when none of the state conditions are satisfied Test: Adding logs. Existing tests pass. Bug: 203971802 Change-Id: Idfea1e19c56d81a7bb74b291f4ac09ba072f62dd --- .../server/policy/DeviceStateProviderImpl.java | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/services/core/java/com/android/server/policy/DeviceStateProviderImpl.java b/services/core/java/com/android/server/policy/DeviceStateProviderImpl.java index c9b3597b7a25a..2a65ea2552b3d 100644 --- a/services/core/java/com/android/server/policy/DeviceStateProviderImpl.java +++ b/services/core/java/com/android/server/policy/DeviceStateProviderImpl.java @@ -395,6 +395,10 @@ public final class DeviceStateProviderImpl implements DeviceStateProvider, break; } } + if (newState == INVALID_DEVICE_STATE) { + Slog.e(TAG, "No declared device states match any of the required conditions."); + dumpSensorValues(); + } if (newState != INVALID_DEVICE_STATE && newState != mLastReportedState) { mLastReportedState = newState; @@ -592,6 +596,19 @@ public final class DeviceStateProviderImpl implements DeviceStateProvider, return null; } + @GuardedBy("mLock") + private void dumpSensorValues() { + Slog.i(TAG, "Sensor values:"); + for (Sensor sensor : mLatestSensorEvent.keySet()) { + SensorEvent sensorEvent = mLatestSensorEvent.get(sensor); + if (sensorEvent != null) { + Slog.i(TAG, sensor.getName() + ": " + Arrays.toString(sensorEvent.values)); + } else { + Slog.i(TAG, sensor.getName() + ": null"); + } + } + } + /** * Tries to parse the provided file into a {@link DeviceStateConfig} object. Returns * {@code null} if the file could not be successfully parsed.