Implement the Blink part of the Device Motion API.

Source/modules/device_orientation/WebDeviceMotionDispatcher.cpp

Index: Source/modules/device_orientation/WebDeviceMotionDispatcher.cpp
diff --git a/Source/modules/device_orientation/WebDeviceMotionDispatcher.cpp b/Source/modules/device_orientation/WebDeviceMotionDispatcher.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..f9d518b32dae64ea595946201757a74449e33b06
--- /dev/null
+++ b/Source/modules/device_orientation/WebDeviceMotionDispatcher.cpp
@@ -0,0 +1,122 @@
+/*
+ * Copyright (C) 2013 Google Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ *     * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "WebDeviceMotionDispatcher.h"
+
+#include <public/Platform.h>
+#include <public/WebDeviceMotionData.h>
+#include <public/WebDeviceMotionHandler.h>
+#include "modules/device_orientation/DeviceMotionController.h"
+#include "modules/device_orientation/DeviceMotionData.h"
+
+using WebCore::DeviceMotionData;
+
+namespace WebKit {
+
+// static
+WebDeviceMotionDispatcher& WebDeviceMotionDispatcher::shared()
+{
+    DEFINE_STATIC_LOCAL(WebDeviceMotionDispatcher, webDeviceMotionDispatcher, ());
+    return webDeviceMotionDispatcher;
+}
+
+WebDeviceMotionDispatcher::WebDeviceMotionDispatcher() : m_lastDeviceMotionData(0)
+{
+}
+
+void WebDeviceMotionDispatcher::addController(WebCore::DeviceMotionController* controller)
+{
+    bool wasEmpty = m_controllers.isEmpty();
+    m_controllers.add(controller);
+    if (wasEmpty)
+        startListening();
+}
+
+void WebDeviceMotionDispatcher::removeController(WebCore::DeviceMotionController* controller)
+{
+    m_controllers.remove(controller);
+    if (m_controllers.isEmpty())
+        stopListening();
+}
+
+void WebDeviceMotionDispatcher::startListening()
+{
+    WebDeviceMotionHandler* handler = WebKit::Platform::current()->deviceMotionHandler();
+    if (handler)

[abarth-chromium, 2013/05/06 17:04:17]

Nit: The style we typically use here is to combine these two lines.

[timvolodine, 2013/05/07 15:40:53]

Done.

+        handler->startListeningToDeviceMotion(this);
+}
+
+void WebDeviceMotionDispatcher::stopListening()
+{
+    WebDeviceMotionHandler* handler = WebKit::Platform::current()->deviceMotionHandler();
+    if (handler)
+        handler->stopListeningToDeviceMotion();
+}
+
+void WebDeviceMotionDispatcher::didChangeDeviceMotion(WebDeviceMotionData& motion)
+{
+    m_lastDeviceMotionData = convertToDeviceMotionData(motion);
+
+    Vector<RefPtr<WebCore::DeviceMotionController> > listenerVector;
+    copyToVector(m_controllers, listenerVector);
+    for (size_t i = 0; i<listenerVector.size(); ++i)
+        listenerVector[i]->didChangeDeviceMotion(m_lastDeviceMotionData.get());
+}
+
+PassRefPtr<DeviceMotionData> WebDeviceMotionDispatcher::convertToDeviceMotionData(WebDeviceMotionData& motion)
+{
+    return DeviceMotionData::create(
+        DeviceMotionData::Acceleration::create(
+            motion.hasAccelerationX(), motion.accelerationX(),
+            motion.hasAccelerationY(), motion.accelerationY(),
+            motion.hasAccelerationZ(), motion.accelerationZ()),
+        DeviceMotionData::Acceleration::create(
+            motion.hasAccelerationIncludingGravityX(), motion.accelerationIncludingGravityX(),
+            motion.hasAccelerationIncludingGravityY(), motion.accelerationIncludingGravityY(),
+            motion.hasAccelerationIncludingGravityZ(), motion.accelerationIncludingGravityZ()),
+        DeviceMotionData::RotationRate::create(
+            motion.hasRotationRateAlpha(), motion.rotationRateAlpha(),
+            motion.hasRotationRateBeta(), motion.rotationRateBeta(),
+            motion.hasRotationRateGamma(), motion.rotationRateGamma()),
+        true, motion.interval());
+}

[abarth-chromium, 2013/05/06 17:04:17]

It's more idiomatic to put this code in a WEBKIT_IMPLEMENTATION function on
WebDeviceMotionData.

[timvolodine, 2013/05/07 15:40:53]

But this way the WebDeviceMotionData (in Platform/chromium/public) is
independent from the Blink/device_orientation module..

+
+
+WebCore::DeviceMotionData* WebDeviceMotionDispatcher::latestDeviceMotionData()
+{
+    return m_lastDeviceMotionData.get();
+}
+
+WebDeviceMotionDispatcher::~WebDeviceMotionDispatcher()
+{
+}
+
+}  // namespace WebKit