Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1647)

Unified Diff: chrome/browser/extensions/api/storage/policy_value_store_unittest.cc

Issue 2257113002: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/storage/policy_value_store_unittest.cc
diff --git a/chrome/browser/extensions/api/storage/policy_value_store_unittest.cc b/chrome/browser/extensions/api/storage/policy_value_store_unittest.cc
index 6b9e759880d2a087fb02fc868a6e787e57e4b6f6..3efd1bcdd905ee859c4bb61f05c5deebab451fd8 100644
--- a/chrome/browser/extensions/api/storage/policy_value_store_unittest.cc
+++ b/chrome/browser/extensions/api/storage/policy_value_store_unittest.cc
@@ -51,8 +51,8 @@ class MutablePolicyValueStore : public PolicyValueStore {
: PolicyValueStore(
kTestExtensionId,
make_scoped_refptr(new SettingsObserverList()),
- base::WrapUnique(
- new LeveldbValueStore(kDatabaseUMAClientName, path))) {}
+ base::MakeUnique<LeveldbValueStore>(kDatabaseUMAClientName, path)) {
+ }
~MutablePolicyValueStore() override {}
WriteResult Set(WriteOptions options,
@@ -103,8 +103,8 @@ class PolicyValueStoreTest : public testing::Test {
observers_->AddObserver(&observer_);
store_.reset(new PolicyValueStore(
kTestExtensionId, observers_,
- base::WrapUnique(new LeveldbValueStore(kDatabaseUMAClientName,
- scoped_temp_dir_.path()))));
+ base::MakeUnique<LeveldbValueStore>(kDatabaseUMAClientName,
+ scoped_temp_dir_.path())));
}
void TearDown() override {
@@ -129,7 +129,7 @@ TEST_F(PolicyValueStoreTest, DontProvideRecommendedPolicies) {
expected.CreateDeepCopy(), nullptr);
policies.Set("may", policy::POLICY_LEVEL_RECOMMENDED,
policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD,
- base::WrapUnique(new base::FundamentalValue(456)), nullptr);
+ base::MakeUnique<base::FundamentalValue>(456), nullptr);
store_->SetCurrentPolicy(policies);
ValueStore::ReadResult result = store_->Get();
ASSERT_TRUE(result->status().ok());

Powered by Google App Engine
This is Rietveld 408576698