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

Unified Diff: services/device/device_service.cc

Issue 2692993006: Port device_sensors to be hosted in Device Service (Closed)
Patch Set: code rebase Created 3 years, 9 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
« no previous file with comments | « services/device/device_service.h ('k') | services/device/manifest.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/device/device_service.cc
diff --git a/services/device/device_service.cc b/services/device/device_service.cc
index f9bd085c75ac674dbeb17aecd383a8b1cc2510ea..a01f1f918cd0f8f6f2319a2a3bcd74f894ffd2ee 100644
--- a/services/device/device_service.cc
+++ b/services/device/device_service.cc
@@ -9,6 +9,7 @@
#include "base/memory/weak_ptr.h"
#include "base/single_thread_task_runner.h"
#include "base/threading/thread_task_runner_handle.h"
+#include "device/sensors/device_sensor_host.h"
#include "services/device/fingerprint/fingerprint.h"
#include "services/device/power_monitor/power_monitor_message_broadcaster.h"
#include "services/device/time_zone_monitor/time_zone_monitor.h"
@@ -49,6 +50,10 @@ void DeviceService::OnStart() {}
bool DeviceService::OnConnect(const service_manager::ServiceInfo& remote_info,
service_manager::InterfaceRegistry* registry) {
registry->AddInterface<mojom::Fingerprint>(this);
+ registry->AddInterface<mojom::LightSensor>(this);
+ registry->AddInterface<mojom::MotionSensor>(this);
+ registry->AddInterface<mojom::OrientationSensor>(this);
+ registry->AddInterface<mojom::OrientationAbsoluteSensor>(this);
registry->AddInterface<mojom::PowerMonitor>(this);
registry->AddInterface<mojom::ScreenOrientationListener>(this);
registry->AddInterface<mojom::TimeZoneMonitor>(this);
@@ -62,6 +67,72 @@ void DeviceService::Create(const service_manager::Identity& remote_identity,
}
void DeviceService::Create(const service_manager::Identity& remote_identity,
+ mojom::LightSensorRequest request) {
+#if defined(OS_ANDROID)
+ // On Android the device sensors implementations need to run on the UI thread
+ // to communicate to Java.
+ DeviceLightHost::Create(std::move(request));
+#else
+ // On platforms other than Android the device sensors implementations run on
+ // the IO thread.
+ if (io_task_runner_) {
+ io_task_runner_->PostTask(FROM_HERE, base::Bind(&DeviceLightHost::Create,
+ base::Passed(&request)));
+ }
+#endif // defined(OS_ANDROID)
+}
+
+void DeviceService::Create(const service_manager::Identity& remote_identity,
+ mojom::MotionSensorRequest request) {
+#if defined(OS_ANDROID)
+ // On Android the device sensors implementations need to run on the UI thread
+ // to communicate to Java.
+ DeviceMotionHost::Create(std::move(request));
+#else
+ // On platforms other than Android the device sensors implementations run on
+ // the IO thread.
+ if (io_task_runner_) {
+ io_task_runner_->PostTask(FROM_HERE, base::Bind(&DeviceMotionHost::Create,
+ base::Passed(&request)));
+ }
+#endif // defined(OS_ANDROID)
+}
+
+void DeviceService::Create(const service_manager::Identity& remote_identity,
+ mojom::OrientationSensorRequest request) {
+#if defined(OS_ANDROID)
+ // On Android the device sensors implementations need to run on the UI thread
+ // to communicate to Java.
+ DeviceOrientationHost::Create(std::move(request));
+#else
+ // On platforms other than Android the device sensors implementations run on
+ // the IO thread.
+ if (io_task_runner_) {
+ io_task_runner_->PostTask(
+ FROM_HERE,
+ base::Bind(&DeviceOrientationHost::Create, base::Passed(&request)));
+ }
+#endif // defined(OS_ANDROID)
+}
+
+void DeviceService::Create(const service_manager::Identity& remote_identity,
+ mojom::OrientationAbsoluteSensorRequest request) {
+#if defined(OS_ANDROID)
+ // On Android the device sensors implementations need to run on the UI thread
+ // to communicate to Java.
+ DeviceOrientationAbsoluteHost::Create(std::move(request));
+#else
+ // On platforms other than Android the device sensors implementations run on
+ // the IO thread.
+ if (io_task_runner_) {
+ io_task_runner_->PostTask(FROM_HERE,
+ base::Bind(&DeviceOrientationAbsoluteHost::Create,
+ base::Passed(&request)));
+ }
+#endif // defined(OS_ANDROID)
+}
+
+void DeviceService::Create(const service_manager::Identity& remote_identity,
mojom::PowerMonitorRequest request) {
if (!power_monitor_message_broadcaster_) {
power_monitor_message_broadcaster_ =
« no previous file with comments | « services/device/device_service.h ('k') | services/device/manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698