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

Unified Diff: device/generic_sensor/platform_sensor_mac.cc

Issue 2332903002: [sensors] [mac] Implement ambient light sensor for macOS (Closed)
Patch Set: More style fixes and build fixes 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_mac.cc
diff --git a/device/generic_sensor/platform_sensor_mac.cc b/device/generic_sensor/platform_sensor_mac.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2ee14cc2281fd0b969497d662c10c4cdc7fe05b9
--- /dev/null
+++ b/device/generic_sensor/platform_sensor_mac.cc
@@ -0,0 +1,50 @@
+// 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/platform_sensor_mac.h"
+
+#include "base/message_loop/message_loop.h"
+
+namespace device {
+
+PlatformSensorMac::PlatformSensorMac(
+ mojom::SensorType type,
+ mojo::ScopedSharedBufferMapping mapping,
+ PlatformSensorProvider* provider,
+ scoped_refptr<base::SingleThreadTaskRunner> polling_thread_task_runner)
+ : PlatformSensor(type, std::move(mapping), provider),
+ polling_thread_task_runner_(polling_thread_task_runner),
Mikhail 2016/09/19 06:25:18 nit: polling_thread_task_runner_(std::move(polling
+ ui_task_runner_(base::MessageLoop::current()->task_runner()) {}
+
+PlatformSensorMac::~PlatformSensorMac() = default;
+
+bool PlatformSensorMac::StartSensor(
+ const PlatformSensorConfiguration& configuration) {
+ polling_thread_task_runner_->PostTask(
+ FROM_HERE,
+ base::Bind(&PlatformSensorMac::BeginPoll, this, configuration));
+ return true;
+}
+
+void PlatformSensorMac::BeginPoll(
+ const PlatformSensorConfiguration& configuration) {
+ DCHECK(polling_thread_task_runner_->BelongsToCurrentThread());
+ timer_.reset(new base::RepeatingTimer());
+ timer_->Start(FROM_HERE, base::TimeDelta::FromMicroseconds(
+ base::Time::kMicrosecondsPerSecond /
+ configuration.frequency()),
+ this, &PlatformSensorMac::UpdateReading);
+}
+
+void PlatformSensorMac::StopSensor() {
+ polling_thread_task_runner_->PostTask(
+ FROM_HERE, base::Bind(&PlatformSensorMac::StopPoll, this));
+}
+
+void PlatformSensorMac::StopPoll() {
+ DCHECK(polling_thread_task_runner_->BelongsToCurrentThread());
+ timer_.reset();
+}
+
+} // namespace device

Powered by Google App Engine
This is Rietveld 408576698