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): figure out how to modify this method in case of
+// multiple event types.
 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();
+  Stop(DeviceData::kTypeOrientation);

[Peter Beverloo, 2013/03/12 11:26:39]

nit: add a TODO to cover that this should shut down the activated data types,
not just orientation.

[timvolodine, 2013/03/12 16:12:46]

Done.

 }
 
 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) {
+  // TODO(timvolodine): copy into shared memory buffer

[Peter Beverloo, 2013/03/12 11:26:39]

It's fine to get rid of the TODO now that there's a NOTIMPLEMENTED() call.

[timvolodine, 2013/03/12 16:12:46]

Done.

+  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(),
                                       device_orientation_.obj(),
                                       reinterpret_cast<jint>(this),
+                                      event_type,

[Peter Beverloo, 2013/03/12 11:26:39]

nit: don't we need a cast here for strictness?  Please run the try-bots :-).

[timvolodine, 2013/03/12 16:12:46]

rate_in_milliseconds below does not use any casts, so I was originally basing on
that.. will check with the bots
On 2013/03/12 11:26:39, Peter Beverloo wrote:

                                       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(),
+                               event_type);

[Peter Beverloo, 2013/03/12 11:26:39]

nit: dito.

[timvolodine, 2013/03/12 16:12:46]

Done.

 }
 
 }  // namespace content