Changes ProviderImpl to use a single DataFetcherFactory

content/browser/device_orientation/provider_unittest.cc

Index: content/browser/device_orientation/provider_unittest.cc
diff --git a/content/browser/device_orientation/provider_unittest.cc b/content/browser/device_orientation/provider_unittest.cc
index 1cb74c78485e517ebb5462600329a373337e8aa3..66e62047397fafdfe1189bfbb61f02b4a62278ee 100644
--- a/content/browser/device_orientation/provider_unittest.cc
+++ b/content/browser/device_orientation/provider_unittest.cc
@@ -165,17 +165,10 @@ class DeviceOrientationProviderTest : public testing::Test {
   }
 
   // Initialize the test fixture with a ProviderImpl that uses the
-  // DataFetcherFactories in the null-terminated factories array.
-  void Init(ProviderImpl::DataFetcherFactory* factories) {
-    provider_ = new ProviderImpl(factories);
-    Provider::SetInstanceForTests(provider_);
-  }
-
-  // Initialize the test fixture with a ProviderImpl that uses the
   // DataFetcherFactory factory.
   void Init(ProviderImpl::DataFetcherFactory factory) {
-    ProviderImpl::DataFetcherFactory factories[] = { factory, NULL };
-    Init(factories);
+    provider_ = new ProviderImpl(&factory);

[hans, 2012/08/01 19:08:47]

here too i think it's just the value that should be passed in, not a pointer to
it

[aousterh, 2012/08/02 10:10:38]

Done.

+    Provider::SetInstanceForTests(provider_);
   }
 
  protected: