Add |notify_client_on_reading_change| flag to sensor configuration

device/generic_sensor/sensor_impl.cc

Index: device/generic_sensor/sensor_impl.cc
diff --git a/device/generic_sensor/sensor_impl.cc b/device/generic_sensor/sensor_impl.cc
index 33e9cd0bb2a0f14e81f6cbcdaedfbe5e4c4da948..9b03167138e58d0dfd19e303d48d85da037faf11 100644
--- a/device/generic_sensor/sensor_impl.cc
+++ b/device/generic_sensor/sensor_impl.cc
@@ -11,7 +11,9 @@
 namespace device {
 
 SensorImpl::SensorImpl(scoped_refptr<PlatformSensor> sensor)
-    : sensor_(std::move(sensor)), suspended_(false) {
+    : sensor_(std::move(sensor)),
+      suspended_(false),
+      not_suppress_on_change_events_count_(0) {
   sensor_->AddClient(this);
 }
 
@@ -28,7 +30,10 @@ void SensorImpl::AddConfiguration(
     AddConfigurationCallback callback) {
   // TODO(Mikhail): To avoid overflowing browser by repeated AddConfigs
   // (maybe limit the number of configs per client).
-  std::move(callback).Run(sensor_->StartListening(this, configuration));
+  bool flag = sensor_->StartListening(this, configuration);

[Reilly Grant (use Gerrit), 2017/06/09 22:10:02]

s/flag/success/

[juncai, 2017/06/09 22:52:32]

Done.

+  if (flag && !configuration.suppress_on_change_events())
+    ++not_suppress_on_change_events_count_;
+  std::move(callback).Run(flag);
 }
 
 void SensorImpl::GetDefaultConfiguration(
@@ -39,7 +44,10 @@ void SensorImpl::GetDefaultConfiguration(
 void SensorImpl::RemoveConfiguration(
     const PlatformSensorConfiguration& configuration,
     RemoveConfigurationCallback callback) {
-  std::move(callback).Run(sensor_->StopListening(this, configuration));
+  bool flag = sensor_->StopListening(this, configuration);
+  if (flag && !configuration.suppress_on_change_events())
+    --not_suppress_on_change_events_count_;
+  std::move(callback).Run(flag);
 }
 
 void SensorImpl::Suspend() {
@@ -54,7 +62,7 @@ void SensorImpl::Resume() {
 
 void SensorImpl::OnSensorReadingChanged() {
   DCHECK(!suspended_);
-  if (client_)
+  if (client_ && not_suppress_on_change_events_count_ > 0)
     client_->SensorReadingChanged();
 }