From e653b95e8f45674da62b328cc8b85ea12f141646 Mon Sep 17 00:00:00 2001 From: Lucas Dupin Date: Tue, 26 Feb 2019 14:00:52 -0800 Subject: Do not unregister sensors after a pulse This generates unnecessary traffic to the system process and SLPI. mDozeSensors.setListening(true) already makes sure that any TriggerSensors will re re-registered. Fixes: 126415281 Test: manual, look at logs Change-Id: Ic7eea1f9c3dd1248dbb5fa38add70c1c2482120c --- packages/SystemUI/src/com/android/systemui/doze/DozeSensors.java | 9 --------- .../SystemUI/src/com/android/systemui/doze/DozeTriggers.java | 3 --- 2 files changed, 12 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/doze/DozeSensors.java b/packages/SystemUI/src/com/android/systemui/doze/DozeSensors.java index 2d1dba6f79c8..1a87ab8b717e 100644 --- a/packages/SystemUI/src/com/android/systemui/doze/DozeSensors.java +++ b/packages/SystemUI/src/com/android/systemui/doze/DozeSensors.java @@ -179,15 +179,6 @@ public class DozeSensors { } } - public void reregisterAllSensors() { - for (TriggerSensor s : mSensors) { - s.setListening(false); - } - for (TriggerSensor s : mSensors) { - s.setListening(true); - } - } - public void onUserSwitched() { for (TriggerSensor s : mSensors) { s.updateListener(); diff --git a/packages/SystemUI/src/com/android/systemui/doze/DozeTriggers.java b/packages/SystemUI/src/com/android/systemui/doze/DozeTriggers.java index 70bf903cd712..b5a3d75cb713 100644 --- a/packages/SystemUI/src/com/android/systemui/doze/DozeTriggers.java +++ b/packages/SystemUI/src/com/android/systemui/doze/DozeTriggers.java @@ -233,9 +233,6 @@ public class DozeTriggers implements DozeMachine.Part { case DOZE: case DOZE_AOD: mDozeSensors.setProxListening(newState != DozeMachine.State.DOZE); - if (oldState != DozeMachine.State.INITIALIZED) { - mDozeSensors.reregisterAllSensors(); - } mDozeSensors.setListening(true); if (newState == DozeMachine.State.DOZE_AOD && !sWakeDisplaySensorState) { onWakeScreen(false, newState); -- cgit v1.2.3-59-g8ed1b