[DeviceLight] Browser+java part

content/browser/device_sensors/device_light_message_filter.cc

Index: content/browser/device_sensors/device_light_message_filter.cc
diff --git a/content/browser/device_sensors/device_light_message_filter.cc b/content/browser/device_sensors/device_light_message_filter.cc
new file mode 100644
index 0000000000000000000000000000000000000000..251b9852293b43479b6f4a429f0e0a5e7c747f54
--- /dev/null
+++ b/content/browser/device_sensors/device_light_message_filter.cc
@@ -0,0 +1,59 @@
+// Copyright 2014 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 "content/browser/device_sensors/device_light_message_filter.h"
+
+#include "content/browser/device_sensors/device_inertial_sensor_service.h"
+#include "content/common/device_sensors/device_light_messages.h"
+
+namespace content {
+
+DeviceLightMessageFilter::DeviceLightMessageFilter()
+    : BrowserMessageFilter(DeviceLightMsgStart), is_started_(false) {

[timvolodine, 2014/09/04 17:16:57]

I think multiline is preferred here, see
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Constructor_In...

[riju_, 2014/09/08 09:26:17]

1. git cl format tells me to do this way.
2. From the style guide, since it fits in 1 line, this way is ok, When it
requires multiple lines, indent 4 spaces... 

+}
+
+DeviceLightMessageFilter::~DeviceLightMessageFilter() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  if (is_started_)

[timvolodine, 2014/09/04 17:16:57]

use braces because multiple lines

[riju_, 2014/09/08 09:26:17]

Done.

+    DeviceInertialSensorService::GetInstance()->RemoveConsumer(
+        CONSUMER_TYPE_LIGHT);
+}
+
+bool DeviceLightMessageFilter::OnMessageReceived(const IPC::Message& message) {
+  bool handled = true;
+  IPC_BEGIN_MESSAGE_MAP(DeviceLightMessageFilter, message)
+  IPC_MESSAGE_HANDLER(DeviceLightHostMsg_StartPolling,
+                      OnDeviceLightStartPolling)
+  IPC_MESSAGE_HANDLER(DeviceLightHostMsg_StopPolling, OnDeviceLightStopPolling)
+  IPC_MESSAGE_UNHANDLED(handled = false)
+  IPC_END_MESSAGE_MAP()
+  return handled;
+}
+
+void DeviceLightMessageFilter::OnDeviceLightStartPolling() {
+  DCHECK(!is_started_);
+  if (is_started_)
+    return;
+  is_started_ = true;
+  DeviceInertialSensorService::GetInstance()->AddConsumer(CONSUMER_TYPE_LIGHT);
+  DidStartDeviceLightPolling();
+}
+
+void DeviceLightMessageFilter::OnDeviceLightStopPolling() {
+  DCHECK(is_started_);
+  if (!is_started_)
+    return;
+  is_started_ = false;
+  DeviceInertialSensorService::GetInstance()->RemoveConsumer(
+      CONSUMER_TYPE_LIGHT);
+}
+
+void DeviceLightMessageFilter::DidStartDeviceLightPolling() {
+  Send(new DeviceLightMsg_DidStartPolling(
+      DeviceInertialSensorService::GetInstance()
+          ->GetSharedMemoryHandleForProcess(CONSUMER_TYPE_LIGHT,
+                                            PeerHandle())));
+}
+
+}  // namespace content