Add device location reporting

content/browser/geolocation/geolocation_provider_unittest.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 "base/memory/singleton.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/message_loop.h"
 #include "base/synchronization/waitable_event.h"
 #include "content/browser/geolocation/arbitrator_dependency_factories_for_test.h"
 #include "content/browser/geolocation/fake_access_token_store.h"
 #include "content/browser/geolocation/geolocation_provider.h"
 #include "content/browser/geolocation/location_arbitrator.h"
 #include "content/browser/geolocation/mock_location_provider.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/test/test_browser_thread.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::AccessTokenStore;
 using content::FakeAccessTokenStore;
 using testing::_;
 using testing::DoAll;
 using testing::DoDefault;
 using testing::Invoke;
 using testing::InvokeWithoutArgs;
 
 namespace {
 
+// A GeolocationProvider that can be constructed and destructed for testing.
+class NonSingletonGeolocationProvider : public GeolocationProvider {
+ public:
+  NonSingletonGeolocationProvider() : GeolocationProvider() { }
+
+  virtual ~NonSingletonGeolocationProvider() {}
+};
+
 class GeolocationProviderTest : public testing::Test {
  protected:
   GeolocationProviderTest()
-      : provider_(new GeolocationProvider),
+      : io_thread_(content::BrowserThread::IO, &message_loop_),
+        provider_(new NonSingletonGeolocationProvider),
         dependency_factory_(
             new GeolocationArbitratorDependencyFactoryWithLocationProvider(
                 &NewAutoSuccessMockNetworkLocationProvider))
   {
   }
 
-  ~GeolocationProviderTest() {
-    DefaultSingletonTraits<GeolocationProvider>::Delete(provider_);
-  }
-
   // testing::Test
   virtual void SetUp() {
     GeolocationArbitrator::SetDependencyFactoryForTest(
         dependency_factory_.get());
   }
 
   // testing::Test
   virtual void TearDown() {
     provider_->Stop();
     GeolocationArbitrator::SetDependencyFactoryForTest(NULL);
   }
 
-  // Message loop for main thread, as provider depends on it existing.
+  // The GeolocationProvider expects to be instantiated on the IO thread.
   MessageLoop message_loop_;
+  content::TestBrowserThread io_thread_;
 
-  // Object under tests. Owned, but not a scoped_ptr due to specialized
-  // destruction protocol.
-  GeolocationProvider* provider_;
+  scoped_ptr<NonSingletonGeolocationProvider> provider_;
 
   scoped_refptr<GeolocationArbitratorDependencyFactory> dependency_factory_;
 };
 
 // Regression test for http://crbug.com/59377
 TEST_F(GeolocationProviderTest, OnPermissionGrantedWithoutObservers) {
   EXPECT_FALSE(provider_->HasPermissionBeenGranted());
   provider_->OnPermissionGranted();
   EXPECT_TRUE(provider_->HasPermissionBeenGranted());
 }
@@ skipping 106 matching lines @@
   provider_->RemoveObserver(&null_observer);
   // Wait for the providers to be stopped.
   event.Wait();
   event.Reset();
   EXPECT_TRUE(provider_->IsRunning());
 
   GeolocationArbitrator::SetDependencyFactoryForTest(NULL);
 }
 
 }  // namespace