Index: chrome/browser/chromeos/policy/device_local_account_browsertest.cc |
diff --git a/chrome/browser/chromeos/policy/device_local_account_browsertest.cc b/chrome/browser/chromeos/policy/device_local_account_browsertest.cc |
index 39ab339807f54843ab9b8d62684ccc82f7d0a542..ff2bdb0de7c1c6fc2982ea59fdaaab46e77de5c0 100644 |
--- a/chrome/browser/chromeos/policy/device_local_account_browsertest.cc |
+++ b/chrome/browser/chromeos/policy/device_local_account_browsertest.cc |
@@ -5,7 +5,9 @@ |
#include "chrome/browser/chromeos/policy/device_local_account.h" |
#include <stddef.h> |
+ |
#include <map> |
+#include <memory> |
#include <set> |
#include <string> |
#include <utility> |
@@ -24,8 +26,8 @@ |
#include "base/json/json_writer.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/message_loop/message_loop.h" |
#include "base/path_service.h" |
#include "base/run_loop.h" |
@@ -233,7 +235,7 @@ class TestingUpdateManifestProvider |
// This method must be registered with the test's EmbeddedTestServer to start |
// serving update manifests. |
- scoped_ptr<net::test_server::HttpResponse> HandleRequest( |
+ std::unique_ptr<net::test_server::HttpResponse> HandleRequest( |
const net::test_server::HttpRequest& request); |
private: |
@@ -305,13 +307,13 @@ void TestingUpdateManifestProvider::AddUpdate(const std::string& id, |
updates_[id] = Update(version, crx_url); |
} |
-scoped_ptr<net::test_server::HttpResponse> |
+std::unique_ptr<net::test_server::HttpResponse> |
TestingUpdateManifestProvider::HandleRequest( |
const net::test_server::HttpRequest& request) { |
base::AutoLock auto_lock(lock_); |
const GURL url("http://localhost" + request.relative_url); |
if (url.path() != relative_update_url_) |
- return scoped_ptr<net::test_server::HttpResponse>(); |
+ return std::unique_ptr<net::test_server::HttpResponse>(); |
std::string content = kUpdateManifestHeader; |
for (net::QueryIterator it(url); !it.IsAtEnd(); it.Advance()) { |
@@ -331,8 +333,8 @@ TestingUpdateManifestProvider::HandleRequest( |
} |
} |
content += kUpdateManifestFooter; |
- scoped_ptr<net::test_server::BasicHttpResponse> |
- http_response(new net::test_server::BasicHttpResponse); |
+ std::unique_ptr<net::test_server::BasicHttpResponse> http_response( |
+ new net::test_server::BasicHttpResponse); |
http_response->set_code(net::HTTP_OK); |
http_response->set_content(content); |
http_response->set_content_type("text/xml"); |
@@ -383,7 +385,7 @@ bool DoesInstallFailureReferToId(const std::string& id, |
.find(base::UTF8ToUTF16(id)) != base::string16::npos; |
} |
-scoped_ptr<net::FakeURLFetcher> RunCallbackAndReturnFakeURLFetcher( |
+std::unique_ptr<net::FakeURLFetcher> RunCallbackAndReturnFakeURLFetcher( |
scoped_refptr<base::SequencedTaskRunner> task_runner, |
const base::Closure& callback, |
const GURL& url, |
@@ -392,8 +394,8 @@ scoped_ptr<net::FakeURLFetcher> RunCallbackAndReturnFakeURLFetcher( |
net::HttpStatusCode response_code, |
net::URLRequestStatus::Status status) { |
task_runner->PostTask(FROM_HERE, callback); |
- return make_scoped_ptr(new net::FakeURLFetcher( |
- url, delegate, response_data, response_code, status)); |
+ return base::WrapUnique(new net::FakeURLFetcher(url, delegate, response_data, |
+ response_code, status)); |
} |
bool IsSessionStarted() { |
@@ -425,7 +427,7 @@ class DeviceLocalAccountTest : public DevicePolicyCrosBrowserTest, |
void SetUp() override { |
// Configure and start the test server. |
- scoped_ptr<crypto::RSAPrivateKey> signing_key( |
+ std::unique_ptr<crypto::RSAPrivateKey> signing_key( |
PolicyBuilder::CreateTestSigningKey()); |
ASSERT_TRUE(test_server_.SetSigningKeyAndSignature( |
signing_key.get(), PolicyBuilder::GetTestSigningKeySignature())); |
@@ -789,7 +791,7 @@ class DeviceLocalAccountTest : public DevicePolicyCrosBrowserTest, |
std::string initial_locale_; |
std::string initial_language_; |
- scoped_ptr<base::RunLoop> run_loop_; |
+ std::unique_ptr<base::RunLoop> run_loop_; |
UserPolicyBuilder device_local_account_policy_; |
LocalPolicyTestServer test_server_; |
@@ -1227,15 +1229,15 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, ExtensionsCached) { |
EXPECT_FALSE(cache->GetExtension(kGoodExtensionID, NULL, NULL)); |
} |
-static void OnPutExtension(scoped_ptr<base::RunLoop>* run_loop, |
- const base::FilePath& file_path, |
- bool file_ownership_passed) { |
+static void OnPutExtension(std::unique_ptr<base::RunLoop>* run_loop, |
+ const base::FilePath& file_path, |
+ bool file_ownership_passed) { |
ASSERT_TRUE(*run_loop); |
(*run_loop)->Quit(); |
} |
static void OnExtensionCacheImplInitialized( |
- scoped_ptr<base::RunLoop>* run_loop) { |
+ std::unique_ptr<base::RunLoop>* run_loop) { |
ASSERT_TRUE(*run_loop); |
(*run_loop)->Quit(); |
} |
@@ -1270,9 +1272,9 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, ExtensionCacheImplTest) { |
CreateFile(impl_path.Append( |
extensions::LocalExtensionCache::kCacheReadyFlagFileName), |
0, base::Time::Now()); |
- extensions::ExtensionCacheImpl cache_impl(make_scoped_ptr( |
+ extensions::ExtensionCacheImpl cache_impl(base::WrapUnique( |
new extensions::ChromeOSExtensionCacheDelegate(impl_path))); |
- scoped_ptr<base::RunLoop> run_loop; |
+ std::unique_ptr<base::RunLoop> run_loop; |
run_loop.reset(new base::RunLoop); |
cache_impl.Start(base::Bind(&OnExtensionCacheImplInitialized, &run_loop)); |
run_loop->Run(); |
@@ -1353,8 +1355,8 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, ExternalData) { |
WaitForPolicy(); |
// Start serving external data at |kExternalDataURL|. |
- scoped_ptr<base::RunLoop> run_loop(new base::RunLoop); |
- scoped_ptr<net::FakeURLFetcherFactory> fetcher_factory( |
+ std::unique_ptr<base::RunLoop> run_loop(new base::RunLoop); |
+ std::unique_ptr<net::FakeURLFetcherFactory> fetcher_factory( |
new net::FakeURLFetcherFactory( |
NULL, base::Bind(&RunCallbackAndReturnFakeURLFetcher, |
base::ThreadTaskRunnerHandle::Get(), |
@@ -1365,7 +1367,7 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, ExternalData) { |
net::URLRequestStatus::SUCCESS); |
// Specify an external data reference for the key::kUserAvatarImage policy. |
- scoped_ptr<base::DictionaryValue> metadata = |
+ std::unique_ptr<base::DictionaryValue> metadata = |
test::ConstructExternalDataReference(kExternalDataURL, kExternalData); |
std::string policy; |
base::JSONWriter::Write(*metadata, &policy); |
@@ -1396,7 +1398,7 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, ExternalData) { |
// |kExternalDataURL|, the retrieval should succeed because the data has been |
// cached. |
run_loop.reset(new base::RunLoop); |
- scoped_ptr<std::string> fetched_external_data; |
+ std::unique_ptr<std::string> fetched_external_data; |
policy_entry->external_data_fetcher->Fetch(base::Bind( |
&test::ExternalDataFetchCallback, |
&fetched_external_data, |
@@ -1475,8 +1477,10 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, UserAvatarImage) { |
run_loop_->Run(); |
user_manager::UserManager::Get()->RemoveObserver(this); |
- scoped_ptr<gfx::ImageSkia> policy_image = chromeos::test::ImageLoader( |
- test_dir.Append(chromeos::test::kUserAvatarImage1RelativePath)).Load(); |
+ std::unique_ptr<gfx::ImageSkia> policy_image = |
+ chromeos::test::ImageLoader( |
+ test_dir.Append(chromeos::test::kUserAvatarImage1RelativePath)) |
+ .Load(); |
ASSERT_TRUE(policy_image); |
const user_manager::User* user = |
@@ -1504,7 +1508,7 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, UserAvatarImage) { |
EXPECT_EQ(user_manager::User::USER_IMAGE_EXTERNAL, image_index); |
EXPECT_EQ(saved_image_path.value(), image_path); |
- scoped_ptr<gfx::ImageSkia> saved_image = |
+ std::unique_ptr<gfx::ImageSkia> saved_image = |
chromeos::test::ImageLoader(saved_image_path).Load(); |
ASSERT_TRUE(saved_image); |
@@ -1810,7 +1814,7 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, MultipleRecommendedLocales) { |
ASSERT_TRUE(content::ExecuteScriptAndExtractString(contents_, |
get_locale_list, |
&json)); |
- scoped_ptr<base::Value> value_ptr = base::JSONReader::Read(json); |
+ std::unique_ptr<base::Value> value_ptr = base::JSONReader::Read(json); |
const base::ListValue* locales = NULL; |
ASSERT_TRUE(value_ptr); |
ASSERT_TRUE(value_ptr->GetAsList(&locales)); |
@@ -2386,7 +2390,7 @@ IN_PROC_BROWSER_TEST_P(TermsOfServiceDownloadTest, TermsOfServiceScreen) { |
" observer.observe(screenElement, options);" |
"}", |
&json)); |
- scoped_ptr<base::Value> value_ptr = base::JSONReader::Read(json); |
+ std::unique_ptr<base::Value> value_ptr = base::JSONReader::Read(json); |
const base::DictionaryValue* status = NULL; |
ASSERT_TRUE(value_ptr); |
ASSERT_TRUE(value_ptr->GetAsDictionary(&status)); |