Refactors DeviceOrientation to make it more extensible

content/browser/device_orientation/provider.h

Index: content/browser/device_orientation/provider.h
diff --git a/content/browser/device_orientation/provider.h b/content/browser/device_orientation/provider.h
index 39ec86007db1ff891627086b72caa1e9fa8a463f..2547004ada95057762ae595612170e626ea5f18f 100644
--- a/content/browser/device_orientation/provider.h
+++ b/content/browser/device_orientation/provider.h
@@ -6,23 +6,31 @@
 #define CONTENT_BROWSER_DEVICE_ORIENTATION_PROVIDER_H_
 
 #include "base/memory/ref_counted.h"
+#include "content/browser/device_orientation/device_data.h"
 #include "content/common/content_export.h"
 
 namespace device_orientation {
 
-class Orientation;
-
 class CONTENT_EXPORT Provider : public base::RefCountedThreadSafe<Provider> {
  public:
   class Observer {
    public:
-    // Called when the orientation changes.
+    // Called when device data changes.
     // An Observer must not synchronously call Provider::RemoveObserver
     // or Provider::AddObserver when this is called.
-    virtual void OnOrientationUpdate(const Orientation& orientation) = 0;
+    virtual void OnDeviceDataUpdate(const DeviceData* device_data,
+                                    DeviceData::Type device_data_type) = 0;
+    DeviceData::Type device_data_type() { return device_data_type_; }
 
    protected:
+    Observer(DeviceData::Type device_data_type)
+        : device_data_type_(device_data_type) {
+    }
     virtual ~Observer() {}
+
+   private:
+    // Each Observer observes exactly one type of DeviceData

[hans, 2012/07/30 16:12:52]

nit: period after end of sentence.

[aousterh, 2012/08/03 11:08:14]

Done.

+    DeviceData::Type device_data_type_;
   };
 
   // Returns a pointer to the singleton instance of this class.
@@ -39,8 +47,7 @@ class CONTENT_EXPORT Provider : public base::RefCountedThreadSafe<Provider> {
   // Get the current instance. Used for testing.
   static Provider* GetInstanceForTests();
 
-  // Note: AddObserver may call back synchronously to the observer with
-  // orientation data.
+  // Note: AddObserver may call back synchronously to the observer with data.
   virtual void AddObserver(Observer* observer) = 0;
   virtual void RemoveObserver(Observer* observer) = 0;