Implement java-side browser sensor polling for device motion and device orientation DOM events.

content/browser/device_orientation/data_fetcher_impl_android.cc

 // Copyright (c) 2012 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_orientation/data_fetcher_impl_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/logging.h"
 #include "content/browser/device_orientation/orientation.h"
-#include "jni/DeviceOrientation_jni.h"
+#include "jni/DeviceMotionAndOrientation_jni.h"
 
 using base::android::AttachCurrentThread;
 
 namespace content {
 
 namespace {
 
 // This should match ProviderImpl::kDesiredSamplingIntervalMs.
 // TODO(husky): Make that constant public so we can use it directly.
 const int kPeriodInMilliseconds = 100;
 
 }  // namespace
 
 DataFetcherImplAndroid::DataFetcherImplAndroid() {
   device_orientation_.Reset(
-      Java_DeviceOrientation_getInstance(AttachCurrentThread()));
+      Java_DeviceMotionAndOrientation_getInstance(AttachCurrentThread()));
 }
 
 void DataFetcherImplAndroid::Init(JNIEnv* env) {
   bool result = RegisterNativesImpl(env);
   DCHECK(result);
 }
 
+// TODO(timvolodine): Modify this method to be able to distinguish
+// device motion from orientation.
 DataFetcher* DataFetcherImplAndroid::Create() {
   scoped_ptr<DataFetcherImplAndroid> fetcher(new DataFetcherImplAndroid);
-  if (fetcher->Start(kPeriodInMilliseconds))
+  if (fetcher->Start(DeviceData::kTypeOrientation, kPeriodInMilliseconds))
     return fetcher.release();
 
   LOG(ERROR) << "DataFetcherImplAndroid::Start failed!";
   return NULL;
 }
 
 DataFetcherImplAndroid::~DataFetcherImplAndroid() {
-  Stop();
+  // TODO(timvolodine): Support device motion as well. Only stop
+  // the active event type(s).
+  Stop(DeviceData::kTypeOrientation);
 }
 
 const DeviceData* DataFetcherImplAndroid::GetDeviceData(
     DeviceData::Type type) {
   if (type != DeviceData::kTypeOrientation)
     return NULL;
   return GetOrientation();
 }
 
 const Orientation* DataFetcherImplAndroid::GetOrientation() {
@@ skipping 14 matching lines @@
   base::AutoLock autolock(next_orientation_lock_);
 
   Orientation* orientation = new Orientation();
   orientation->set_alpha(alpha);
   orientation->set_beta(beta);
   orientation->set_gamma(gamma);
   orientation->set_absolute(true);
   next_orientation_ = orientation;
 }
 
-bool DataFetcherImplAndroid::Start(int rate_in_milliseconds) {
+void DataFetcherImplAndroid::GotAcceleration(
+    JNIEnv*, jobject, double x, double y, double z) {
+  NOTIMPLEMENTED();
+}
+
+void DataFetcherImplAndroid::GotAccelerationIncludingGravity(
+    JNIEnv*, jobject, double x, double y, double z) {
+  NOTIMPLEMENTED();
+}
+
+void DataFetcherImplAndroid::GotRotationRate(
+    JNIEnv*, jobject, double alpha, double beta, double gamma) {
+  NOTIMPLEMENTED();
+}
+
+bool DataFetcherImplAndroid::Start(DeviceData::Type event_type,
+    int rate_in_milliseconds) {
   DCHECK(!device_orientation_.is_null());
-  return Java_DeviceOrientation_start(AttachCurrentThread(),
+  return Java_DeviceMotionAndOrientation_start(AttachCurrentThread(),

[bulach, 2013/03/19 15:22:36]

nit: preferred way for callers is to move the first param to the next line
indented by +4, and put as many params per line as possible... (for
declarations, it's better to keep left-aligned as it was...)

[timvolodine, 2013/03/19 16:50:08]

Done.

                                       device_orientation_.obj(),
                                       reinterpret_cast<jint>(this),
+                                      static_cast<jint>(event_type),
                                       rate_in_milliseconds);
 }
 
-void DataFetcherImplAndroid::Stop() {
+void DataFetcherImplAndroid::Stop(DeviceData::Type event_type) {
   DCHECK(!device_orientation_.is_null());
-  Java_DeviceOrientation_stop(AttachCurrentThread(), device_orientation_.obj());
+  Java_DeviceMotionAndOrientation_stop(AttachCurrentThread(),
+                               device_orientation_.obj(),

[bulach, 2013/03/19 15:22:36]

nit: as above..

[timvolodine, 2013/03/19 16:50:08]

Done.

+                               static_cast<jint>(event_type));
 }
 
 }  // namespace content