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

[Peter Beverloo, 2013/03/12 17:11:22]

What does this mean?  Figuring something out does not imply actually making the
change.  The TODO should read "Modify this method to be able to distinguish
device motion from orientation.".

[timvolodine, 2013/03/13 12:32:24]

Done.

+// multiple event types.
 DataFetcher* DataFetcherImplAndroid::Create() {

[Miguel Garcia, 2013/03/12 17:00:26]

don't you want to at least change the interface now and pass an argument with
the types we want? Then on the implementation you can Start only the
kTypeOtrientation and throw unimplemented for the rest.

[timvolodine, 2013/03/13 12:32:24]

This probably goes out of scope of this CL, because this intended to be mainly
java-side.
We'll need to figure out how to best handle both types as the current ::Create()
approach seems not very appropriate anymore.
On 2013/03/12 17:00:26, Miguel Garcia wrote:

   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): stop all active event types, not only

[Peter Beverloo, 2013/03/12 17:11:22]

This TODO should read "Support device motion as well. Only stop the active event
type(s).".

[timvolodine, 2013/03/13 12:32:24]

Done.

+  // device orientation, when implemented.
+  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(),
                                       device_orientation_.obj(),
                                       reinterpret_cast<jint>(this),
+                                      event_type,

[Miguel Garcia, 2013/03/12 17:00:26]

what happened to the cast suggestion from Peter?

[Peter Beverloo, 2013/03/12 17:11:22]

Try-jobs are running, let's see what they say.  Nonetheless, I prefer to just
static_cast<> them to an integer given that I think it's cleaner.

[timvolodine, 2013/03/13 12:32:24]

Done.

[timvolodine, 2013/03/13 12:32:24]

ok, added static_cast<jint>

On 2013/03/12 17:11:22, 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 17:11:22]

Dito to my reply to Miguel's comment on line 104.

[timvolodine, 2013/03/13 12:32:24]

Done.

 }
 
 }  // namespace content