summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Dave Mankoff <mankoff@google.com> 2019-07-24 18:06:44 -0400
committer Dave Mankoff <mankoff@google.com> 2019-07-24 22:12:40 +0000
commit0ae8f2e36a6c86d54585fd57d55e184b0d99de1c (patch)
tree32af81b5ba855612de03b6b3b48cd440320a0c64
parent8d56bbf2e32c96bc515d2d3e4a3a4a17a7b22ce5 (diff)
Prevent the BrightLineFalsingManager from duping its session.
The BrightLineFalsingManager should not be able to start a session when it's already in a session. Primarily, this caused the FalsingManager to hang onto extra registrations to the Proximity Sensor, per the bug. Bug: 138220274 Test: atest SystemUITests and manual. Change-Id: Id10d2697a96524e98c87aaa87702209d1752fe68
-rw-r--r--packages/SystemUI/src/com/android/systemui/classifier/brightline/BrightLineFalsingManager.java10
1 files changed, 6 insertions, 4 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/classifier/brightline/BrightLineFalsingManager.java b/packages/SystemUI/src/com/android/systemui/classifier/brightline/BrightLineFalsingManager.java
index 3d869bcc2e4b..6280c9f7c00f 100644
--- a/packages/SystemUI/src/com/android/systemui/classifier/brightline/BrightLineFalsingManager.java
+++ b/packages/SystemUI/src/com/android/systemui/classifier/brightline/BrightLineFalsingManager.java
@@ -98,10 +98,12 @@ public class BrightLineFalsingManager implements FalsingManager {
}
private void sessionStart() {
- logDebug("Starting Session");
- mSessionStarted = true;
- registerSensors();
- mClassifiers.forEach(FalsingClassifier::onSessionStarted);
+ if (!mSessionStarted) {
+ logDebug("Starting Session");
+ mSessionStarted = true;
+ registerSensors();
+ mClassifiers.forEach(FalsingClassifier::onSessionStarted);
+ }
}
private void sessionEnd() {