Refactors DeviceOrientation to make it more extensible

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/android/scoped_java_ref.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ skipping 36 matching lines @@
     return fetcher.release();
 
   LOG(ERROR) << "DataFetcherImplAndroid::Start failed!";
   return NULL;
 }
 
 DataFetcherImplAndroid::~DataFetcherImplAndroid() {
   Stop();
 }
 
-bool DataFetcherImplAndroid::GetOrientation(Orientation* orientation) {
+DeviceData* DataFetcherImplAndroid::GetDeviceData(
+    DeviceData::DeviceDataType* device_data_type) {
+  switch (device_data_type) {

[bulach, 2012/07/12 10:43:27]

nit: as above..

[aousterh, 2012/07/12 17:13:57]

Actually for this one, I will (hopefully) be adding DeviceMotion soon. I can
change it to the if-else way for now if you'd prefer, but in the long run it
probably makes more sense as a switch-case statement.

+    case DeviceData::kDeviceOrientationData:
+      return GetOrientation();
+    default:
+      return NULL;
+  }
+}
+
+Orientation* DataFetcherImplAndroid::GetOrientation() {
   // Do we have a new orientation value? (It's safe to do this outside the lock
   // because we only skip the lock if the value is null. We always enter the
   // lock if we're going to make use of the new value.)
   if (next_orientation_.get()) {
     base::AutoLock autolock(next_orientation_lock_);
     next_orientation_.swap(current_orientation_);
   }
+  scoped_ptr<Orientation> orientation(new Orientation());
   if (current_orientation_.get())
     *orientation = *current_orientation_;
-  return true;
+  return orientation.release();
 }
 
 void DataFetcherImplAndroid::GotOrientation(
     JNIEnv*, jobject, double alpha, double beta, double gamma) {
   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_.reset(orientation);
 }
 
 bool DataFetcherImplAndroid::Start(int rate_in_milliseconds) {
   DCHECK(!g_jni_obj.Get().is_null());
   return Java_DeviceOrientation_start(AttachCurrentThread(),
                                       g_jni_obj.Get().obj(),
                                       reinterpret_cast<jint>(this),
                                       rate_in_milliseconds);
 }
 
 void DataFetcherImplAndroid::Stop() {
   DCHECK(!g_jni_obj.Get().is_null());
   Java_DeviceOrientation_stop(AttachCurrentThread(), g_jni_obj.Get().obj());
 }
 
 }  // namespace device_orientation