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

Unified Diff: device/generic_sensor/platform_sensor_provider_win.cc

Issue 2447973003: [sensors] [win] Implement ambient light sensor for Windows platform (Closed)
Patch Set: Modify timestamp calculation as proposed by miu@ Created 4 years, 2 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_provider_win.cc
diff --git a/device/generic_sensor/platform_sensor_provider_win.cc b/device/generic_sensor/platform_sensor_provider_win.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3bbc28a8d7718954dba5dca8d28dc5250b326aaf
--- /dev/null
+++ b/device/generic_sensor/platform_sensor_provider_win.cc
@@ -0,0 +1,116 @@
+// 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_provider_win.h"
+
+#include "base/memory/singleton.h"
+#include "base/task_runner_util.h"
+#include "base/threading/thread.h"
+#include "base/win/windows_version.h"
+#include "device/generic_sensor/platform_sensor_win.h"
+
+namespace device {
+
+// static
+PlatformSensorProvider* PlatformSensorProvider::GetInstance() {
+ return PlatformSensorProviderWin::GetInstance();
+}
+
+// static
+PlatformSensorProviderWin* PlatformSensorProviderWin::GetInstance() {
+ if (base::win::GetVersion() < base::win::VERSION_WIN7) {
Lei Zhang 2016/10/29 05:10:41 Modern Chromium only supports Win7 and newer.
shalamov 2016/10/31 13:12:57 I removed check. Thanks.
+ VLOG(1) << "Sensor API is only supported on Windows 7 or newer versions.";
+ return nullptr;
+ }
+
+ return base::Singleton<
+ PlatformSensorProviderWin,
+ base::LeakySingletonTraits<PlatformSensorProviderWin>>::get();
+}
+
+void PlatformSensorProviderWin::SetSensorManagerForTesing(
+ base::win::ScopedComPtr<ISensorManager> sensor_manager) {
+ sensor_manager_ = sensor_manager;
+}
+
+PlatformSensorProviderWin::PlatformSensorProviderWin() = default;
+PlatformSensorProviderWin::~PlatformSensorProviderWin() = default;
+
+void PlatformSensorProviderWin::CreateSensorInternal(
+ mojom::SensorType type,
+ mojo::ScopedSharedBufferMapping mapping,
+ const CreateSensorCallback& callback) {
+ DCHECK(CalledOnValidThread());
+ if (!StartSensorThread()) {
+ callback.Run(nullptr);
+ return;
+ }
+
+ base::PostTaskAndReplyWithResult(
+ sensor_thread_->task_runner().get(), FROM_HERE,
+ base::Bind(&PlatformSensorProviderWin::CreateSensorReader,
+ base::Unretained(this), type),
+ base::Bind(&PlatformSensorProviderWin::SensorReaderCreated,
+ base::Unretained(this), type, base::Passed(&mapping),
+ callback));
+}
+
+bool PlatformSensorProviderWin::InitializeSensorManager() {
+ if (sensor_manager_)
+ return true;
+
+ HRESULT hr = sensor_manager_.CreateInstance(CLSID_SensorManager);
+ return !FAILED(hr) && sensor_manager_;
Lei Zhang 2016/10/29 05:10:42 Why not SUCCEEDED(hr)?
shalamov 2016/10/31 13:12:57 Done.
+}
+
+void PlatformSensorProviderWin::AllSensorsRemoved() {
+ StopSensorThread();
+}
+
+bool PlatformSensorProviderWin::StartSensorThread() {
+ if (!sensor_thread_) {
+ sensor_thread_.reset(new base::Thread("Windows platform sensor thread"));
Lei Zhang 2016/10/29 05:10:41 base::MakeUnique
shalamov 2016/10/31 13:12:57 Done.
+ sensor_thread_->init_com_with_mta(true);
+ }
+
+ if (!sensor_thread_->IsRunning())
+ return sensor_thread_->Start();
+ return true;
+}
+
+void PlatformSensorProviderWin::StopSensorThread() {
+ if (sensor_thread_ && sensor_thread_->IsRunning()) {
+ sensor_manager_.Release();
+ sensor_thread_->Stop();
+ }
+}
+
+void PlatformSensorProviderWin::SensorReaderCreated(
+ mojom::SensorType type,
+ mojo::ScopedSharedBufferMapping mapping,
+ const CreateSensorCallback& callback,
+ std::unique_ptr<PlatformSensorReaderWin> sensor_reader) {
+ DCHECK(CalledOnValidThread());
+ if (!sensor_reader) {
+ callback.Run(nullptr);
+ if (!HasSensors())
+ StopSensorThread();
+ return;
+ }
+
+ scoped_refptr<PlatformSensor> sensor = new PlatformSensorWin(
+ type, std::move(mapping), this, sensor_thread_->task_runner(),
+ std::move(sensor_reader));
+ callback.Run(sensor);
+}
+
+std::unique_ptr<PlatformSensorReaderWin>
+PlatformSensorProviderWin::CreateSensorReader(mojom::SensorType type) {
+ DCHECK(sensor_thread_->task_runner()->BelongsToCurrentThread());
+ if (!InitializeSensorManager())
+ return nullptr;
+ return PlatformSensorReaderWin::Create(type, sensor_manager_);
+}
+
+} // namespace device

Powered by Google App Engine
This is Rietveld 408576698