Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1265)

Unified Diff: device/generic_sensor/sensor_impl.h

Issue 2927263002: Add |notify_client_on_reading_change| flag to sensor configuration (Closed)
Patch Set: fix browser test Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/generic_sensor/sensor_impl.h
diff --git a/device/generic_sensor/sensor_impl.h b/device/generic_sensor/sensor_impl.h
index 4a6ecc95e0faaff7590e15aba65c74ad9b26a0ea..9495f6116a2419ba68808de366c4bfd588c578b3 100644
--- a/device/generic_sensor/sensor_impl.h
+++ b/device/generic_sensor/sensor_impl.h
@@ -40,6 +40,10 @@ class SensorImpl final : public mojom::Sensor, public PlatformSensor::Client {
scoped_refptr<PlatformSensor> sensor_;
mojom::SensorClientPtr client_;
bool suspended_;
+ // The number of configurations that have |notify_client_on_reading_change_|
+ // flag set to true. If there is at least one configuration that sets this
+ // flag to true, SensorClient::SensorReadingChanged() is called.
timvolodine 2017/06/09 12:39:09 Is it possible to only notify the client with the
timvolodine 2017/06/09 12:40:42 hmm scratch that, looks like there is only one cli
+ int notify_client_on_reading_change_count_;
DISALLOW_COPY_AND_ASSIGN(SensorImpl);
};

Powered by Google App Engine
This is Rietveld 408576698