Changes ProviderImpl to use a single DataFetcherFactory

content/browser/device_orientation/provider_impl.h

Index: content/browser/device_orientation/provider_impl.h
diff --git a/content/browser/device_orientation/provider_impl.h b/content/browser/device_orientation/provider_impl.h
index 32c63691a6a531cc9ed313de23ac86aa1ce86115..6bc2bbe2648ea10c00d264caae6f10294f746228 100644
--- a/content/browser/device_orientation/provider_impl.h
+++ b/content/browser/device_orientation/provider_impl.h
@@ -24,9 +24,10 @@ class ProviderImpl : public Provider {
  public:
   typedef DataFetcher* (*DataFetcherFactory)();
 
-  // Create a ProviderImpl that uses the NULL-terminated factories array to find
-  // a DataFetcher that can provide orientation data.
-  CONTENT_EXPORT ProviderImpl(const DataFetcherFactory factories[]);
+  // Create a ProviderImpl that uses the pointer to a factory to create a

[hans, 2012/08/02 10:19:31]

we can just say "uses the factory to create a ..."

+  // DataFetcher that can provide orientation data. A NULL pointer indicates
+  // that there are no DataFetchers for this OS.
+  CONTENT_EXPORT ProviderImpl(DataFetcherFactory factory);
 
   // From Provider.
   virtual void AddObserver(Observer* observer) OVERRIDE;
@@ -52,7 +53,7 @@ class ProviderImpl : public Provider {
   MessageLoop* creator_loop_;
 
   // Members below are only to be used from the creator_loop_.
-  std::vector<DataFetcherFactory> factories_;
+  DataFetcherFactory factory_;
   std::set<Observer*> observers_;
   Orientation last_notification_;