tree 89a7ee33d8c4d9f3dc821d2d024f18254d9951ed
parent c63880a5a946a23ceacb9a8b7bb9c7dcd38d93fa
author Chris Kuiper <ckuiper@google.com> 1628057290 -0700
committer Chris Kuiper <ckuiper@google.com> 1628117327 -0700

sensorservice: ProximitySensor needs to report correct sensor state.

Calls to ProximitySensor::activate() may repeatedly set enabled=true.
The current logic doesn't account for that. In order to correctly
report the current sensor state, simply use
mSensorDevice.isSensorActive() instead.

Bug: 194878856
Test: 1) Reproduced problem and verified mIsProxActive in "dumpsys
         display" contains the correct value and no more 60/90Hz
         flicker.
      2) Adding local debugs, then mixing making phone calls with
         additional "sensor_test sample -s8 -n5" commands to create
         several Prox clients, and verifying logged flags and
         reference counts.
Change-Id: I2342844ff4e6301a6b9ac8a33dc3e6047fca83ad