commit afabca129893472e735a579efca0398f02dbb17b
author Arthur Ishiguro <arthuri@google.com> Wed Jul 18 10:30:24 2018 -0700
committer Arthur Ishiguro <arthuri@google.com> Wed Jul 18 13:07:13 2018 -0700
tree b3074d290d30583b9cd7c559487122dcbb47eef1
parent eec7e6699bcde3638b9976afd75897c836206874

Filter when to send interval/latency status events

Only post interval/latency status updates to nanoapps if the sensor
became enabled. This prevents posting the event to nanoapps in the case
when the sensor was stayed disabled, but received a onSamplingStatusUpdate
callback.

Bug: 111444100
Test: Compile and run sensor world with passive request to gyro,
      verify events are not sent when the sensor was already disabled

Change-Id: Id97d0f59549eedf6caaf30d7a182737581a62e83

platform/slpi/see/platform_sensor.cc

diff --git a/platform/slpi/see/platform_sensor.cc b/platform/slpi/see/platform_sensor.cc
index 857b6ed..5cf6a51 100644
--- a/platform/slpi/see/platform_sensor.cc
+++ b/platform/slpi/see/platform_sensor.cc
@@ -190,9 +190,9 @@
       newStatus.latency = update.status.latency;
     }
 
-    if (newStatus.enabled != prevStatus.enabled
-        || newStatus.interval != prevStatus.interval
-        || newStatus.latency != prevStatus.latency) {
+    if (newStatus.enabled != prevStatus.enabled ||
+        (newStatus.enabled && (newStatus.interval != prevStatus.interval
+                               || newStatus.latency != prevStatus.latency))) {
       sensor->setSamplingStatus(newStatus);
 
       // Only post to Nanoapps with an open request.