diff options
| author | 2019-07-24 18:06:44 -0400 | |
|---|---|---|
| committer | 2019-07-24 22:12:40 +0000 | |
| commit | 0ae8f2e36a6c86d54585fd57d55e184b0d99de1c (patch) | |
| tree | 32af81b5ba855612de03b6b3b48cd440320a0c64 | |
| parent | 8d56bbf2e32c96bc515d2d3e4a3a4a17a7b22ce5 (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.java | 10 |
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() { |