Index: content/browser/geolocation/geolocation_provider_impl_unittest.cc |
diff --git a/content/browser/geolocation/geolocation_provider_impl_unittest.cc b/content/browser/geolocation/geolocation_provider_impl_unittest.cc |
index f1c7323f5a4e01f8e5b960076dcbf71d8d3d0ebe..a2ceaf63ec954e981aea1b0123447c24aa16fe05 100644 |
--- a/content/browser/geolocation/geolocation_provider_impl_unittest.cc |
+++ b/content/browser/geolocation/geolocation_provider_impl_unittest.cc |
@@ -2,16 +2,19 @@ |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
+#include "content/browser/geolocation/geolocation_provider_impl.h" |
+ |
+#include <memory> |
+ |
#include "base/bind.h" |
#include "base/bind_helpers.h" |
#include "base/location.h" |
#include "base/macros.h" |
+#include "base/memory/ptr_util.h" |
#include "base/memory/ref_counted.h" |
-#include "base/memory/scoped_ptr.h" |
#include "base/single_thread_task_runner.h" |
#include "base/strings/string16.h" |
#include "base/time/time.h" |
-#include "content/browser/geolocation/geolocation_provider_impl.h" |
#include "content/browser/geolocation/mock_location_arbitrator.h" |
#include "content/public/browser/access_token_store.h" |
#include "content/public/browser/browser_thread.h" |
@@ -38,18 +41,18 @@ class LocationProviderForTestArbitrator : public GeolocationProviderImpl { |
protected: |
// GeolocationProviderImpl implementation: |
- scoped_ptr<LocationArbitrator> CreateArbitrator() override; |
+ std::unique_ptr<LocationArbitrator> CreateArbitrator() override; |
private: |
// An alias to the arbitrator stored in the super class, where it is owned. |
MockLocationArbitrator* mock_arbitrator_; |
}; |
-scoped_ptr<LocationArbitrator> |
+std::unique_ptr<LocationArbitrator> |
LocationProviderForTestArbitrator::CreateArbitrator() { |
DCHECK(mock_arbitrator_ == NULL); |
mock_arbitrator_ = new MockLocationArbitrator; |
- return make_scoped_ptr(mock_arbitrator_); |
+ return base::WrapUnique(mock_arbitrator_); |
} |
class GeolocationObserver { |
@@ -136,7 +139,7 @@ class GeolocationProviderTest : public testing::Test { |
base::MessageLoop message_loop_; |
TestBrowserThread ui_thread_; |
- scoped_ptr<LocationProviderForTestArbitrator> provider_; |
+ std::unique_ptr<LocationProviderForTestArbitrator> provider_; |
}; |
@@ -179,7 +182,7 @@ TEST_F(GeolocationProviderTest, StartStop) { |
EXPECT_FALSE(provider()->IsRunning()); |
GeolocationProviderImpl::LocationUpdateCallback callback = |
base::Bind(&DummyFunction); |
- scoped_ptr<content::GeolocationProvider::Subscription> subscription = |
+ std::unique_ptr<content::GeolocationProvider::Subscription> subscription = |
provider()->AddLocationUpdateCallback(callback, false); |
EXPECT_TRUE(provider()->IsRunning()); |
EXPECT_TRUE(ProvidersStarted()); |
@@ -202,7 +205,7 @@ TEST_F(GeolocationProviderTest, StalePositionNotSent) { |
&MockGeolocationObserver::OnLocationUpdate, |
base::Unretained(&first_observer)); |
EXPECT_CALL(first_observer, OnLocationUpdate(GeopositionEq(first_position))); |
- scoped_ptr<content::GeolocationProvider::Subscription> subscription = |
+ std::unique_ptr<content::GeolocationProvider::Subscription> subscription = |
provider()->AddLocationUpdateCallback(first_callback, false); |
SendMockLocation(first_position); |
base::MessageLoop::current()->Run(); |
@@ -223,7 +226,7 @@ TEST_F(GeolocationProviderTest, StalePositionNotSent) { |
GeolocationProviderImpl::LocationUpdateCallback second_callback = base::Bind( |
&MockGeolocationObserver::OnLocationUpdate, |
base::Unretained(&second_observer)); |
- scoped_ptr<content::GeolocationProvider::Subscription> subscription2 = |
+ std::unique_ptr<content::GeolocationProvider::Subscription> subscription2 = |
provider()->AddLocationUpdateCallback(second_callback, false); |
base::MessageLoop::current()->RunUntilIdle(); |
@@ -248,7 +251,7 @@ TEST_F(GeolocationProviderTest, OverrideLocationForTesting) { |
GeolocationProviderImpl::LocationUpdateCallback callback = base::Bind( |
&MockGeolocationObserver::OnLocationUpdate, |
base::Unretained(&mock_observer)); |
- scoped_ptr<content::GeolocationProvider::Subscription> subscription = |
+ std::unique_ptr<content::GeolocationProvider::Subscription> subscription = |
provider()->AddLocationUpdateCallback(callback, false); |
subscription.reset(); |
// Wait for the providers to be stopped now that all clients are gone. |