[DeviceLight] Add renderer+common parts

content/renderer/device_sensors/device_light_event_pump.cc

+// 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/renderer/device_sensors/device_light_event_pump.h"
+
+#include "content/common/device_sensors/device_light_messages.h"
+#include "content/public/renderer/render_thread.h"
+#include "third_party/WebKit/public/platform/WebDeviceLightListener.h"
+
+namespace content {
+
+const int DeviceLightEventPump::kDefaultLightPumpDelayMillis = 200;
+
+DeviceLightEventPump::DeviceLightEventPump()
+    : DeviceSensorEventPump(kDefaultLightPumpDelayMillis),

[dcheng, 2014/06/18 16:51:19]

Not for this patch, but it would be nice to fix the underlying class to use
base::TimeDelta here in the long run.

[riju_, 2014/06/19 13:15:54]

Ok. I will put up the new CL once this goes in.

+      listener_(0),
+      last_seen_data_(-1) {

[timvolodine, 2014/06/18 17:38:36]

you probably also want to set to -1 when Stop is called.

[riju_, 2014/06/19 13:15:54]

Done.

+}
+
+DeviceLightEventPump::DeviceLightEventPump(int pump_delay_millis)
+    : DeviceSensorEventPump(pump_delay_millis),
+      listener_(0),

[dcheng, 2014/06/18 16:51:19]

NULL

[riju_, 2014/06/19 13:15:54]

Done.

+      last_seen_data_(-1) {
+}
+
+DeviceLightEventPump::~DeviceLightEventPump() {
+}
+
+bool DeviceLightEventPump::SetListener(
+    blink::WebDeviceLightListener* listener) {
+  listener_ = listener;
+  return listener_ ? RequestStart() : Stop();
+}
+
+bool DeviceLightEventPump::OnControlMessageReceived(
+    const IPC::Message& message) {
+  bool handled = true;
+  IPC_BEGIN_MESSAGE_MAP(DeviceLightEventPump, message)
+  IPC_MESSAGE_HANDLER(DeviceLightMsg_DidStartPolling, OnDidStart)
+  IPC_MESSAGE_UNHANDLED(handled = false)
+  IPC_END_MESSAGE_MAP()
+  return handled;
+}
+
+void DeviceLightEventPump::FireEvent() {
+  DCHECK(listener_);
+  DeviceLightData data;
+  bool did_return_light_data = reader_->GetLatestData(&data);
+  if (data.value != last_seen_data_ && did_return_light_data) {

[dcheng, 2014/06/18 16:51:19]

Nit: It probably makes more sense to order the check for did_return_light_data
first.

[riju_, 2014/06/19 13:15:54]

Done.

+    last_seen_data_ = data.value;
+    listener_->didChangeDeviceLight(data.value);
+  }
+}
+
+bool DeviceLightEventPump::InitializeReader(base::SharedMemoryHandle handle) {
+  if (!reader_)
+    reader_.reset(new DeviceLightSharedMemoryReader());
+  return reader_->Initialize(handle);
+}
+
+bool DeviceLightEventPump::SendStartMessage() {
+  return RenderThread::Get()->Send(new DeviceLightHostMsg_StartPolling());
+}
+
+bool DeviceLightEventPump::SendStopMessage() {
+  return RenderThread::Get()->Send(new DeviceLightHostMsg_StopPolling());
+}
+
+}  // namespace content