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

Unified Diff: device/generic_sensor/mock_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/mock_platform_sensor.cc
diff --git a/device/generic_sensor/mock_platform_sensor.cc b/device/generic_sensor/mock_platform_sensor.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a2fa173564b49f85e507b1256709a22833df74a0
--- /dev/null
+++ b/device/generic_sensor/mock_platform_sensor.cc
@@ -0,0 +1,51 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "device/generic_sensor/mock_platform_sensor.h"
+
+namespace device {
+
+MockPlatformSensor::MockPlatformSensor(mojom::SensorType type,
+ mojo::ScopedSharedBufferMapping mapping,
+ uint64_t buffer_size,
+ PlatformSensorProvider* provider)
+ : PlatformSensor(type, std::move(mapping), provider), started_(false) {}
+
+MockPlatformSensor::~MockPlatformSensor() = default;
+
+mojom::ReportingMode MockPlatformSensor::GetReportingMode() {
+ return mojom::ReportingMode::CONTINUOUS;
+}
+
+PlatformSensorConfiguration MockPlatformSensor::GetDefaultConfiguration() {
+ return PlatformSensorConfiguration();
+}
+
+bool MockPlatformSensor::StartSensor(
+ const PlatformSensorConfiguration& configuration) {
+ config_ = configuration;
+ started_ = true;
+ return started_;
+}
+
+void MockPlatformSensor::StopSensor() {
+ started_ = false;
+}
+
+bool MockPlatformSensor::CheckSensorConfiguration(
+ const PlatformSensorConfiguration& configuration) {
+ if (configuration.frequency() > kMaxFrequencyValue)
shalamov 2016/09/07 09:53:01 return configuration.frequency() <= kMaxFrequencyV
Mikhail 2016/09/07 10:46:19 return configuration.frequency() <= kMaxFrequencyV
maksims (do not use this acc) 2016/09/09 10:39:05 Done.
maksims (do not use this acc) 2016/09/09 10:39:05 Done.
+ return false;
+ return true;
+}
+
+void MockPlatformSensor::NotifySensorReadingChanged() {
Mikhail 2016/09/07 10:46:19 why override this method?
maksims (do not use this acc) 2016/09/09 10:39:05 I cannot access them outside this code. Those meth
+ PlatformSensor::NotifySensorReadingChanged();
+}
+
+void MockPlatformSensor::NotifySensorError() {
Mikhail 2016/09/07 10:46:19 ditto.
+ PlatformSensor::NotifySensorError();
+}
+
+} // namespace device

Powered by Google App Engine
This is Rietveld 408576698