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

Unified Diff: device/generic_sensor/platform_sensor.cc

Issue 2306333002: [sensors] Add Generic Sensor platform unit tests. (Closed)
Patch Set: Generic Sensor Unittest Created 4 years, 3 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/platform_sensor.cc
diff --git a/device/generic_sensor/platform_sensor.cc b/device/generic_sensor/platform_sensor.cc
index 5712abc5f6e61ea8af8140796c1f67a82ab38399..2c7e9f339250596947c6a8872a4581ec7142fdfb 100644
--- a/device/generic_sensor/platform_sensor.cc
+++ b/device/generic_sensor/platform_sensor.cc
@@ -94,4 +94,26 @@ void PlatformSensor::NotifySensorError() {
FOR_EACH_OBSERVER(Client, clients_, OnSensorError());
}
+bool PlatformSensor::UpdateSensorInternal(const ConfigMap& configurations) {
+ const PlatformSensorConfiguration* optimal_configuration = nullptr;
+ for (const auto& pair : configurations) {
+ if (pair.first->IsNotificationSuspended())
+ continue;
+
+ const auto& conf_list = pair.second;
+ for (const auto& configuration : conf_list) {
+ if (!optimal_configuration || configuration > *optimal_configuration) {
+ optimal_configuration = &configuration;
+ }
+ }
+ }
+
+ if (!optimal_configuration) {
+ StopSensor();
+ return true;
+ }
+
+ return StartSensor(*optimal_configuration);
+}
+
} // namespace device

Powered by Google App Engine
This is Rietveld 408576698