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

Unified Diff: chrome/browser/prefs/proxy_policy_unittest.cc

Issue 2334613003: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Changes from review by sky Created 4 years, 3 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/prefs/proxy_policy_unittest.cc
diff --git a/chrome/browser/prefs/proxy_policy_unittest.cc b/chrome/browser/prefs/proxy_policy_unittest.cc
index 285be0d53909fb4ce48d5f6d8a7a390744ffa1e3..0d05a0a8e6073ebfb01fb01e4b1a595623b8d29e 100644
--- a/chrome/browser/prefs/proxy_policy_unittest.cc
+++ b/chrome/browser/prefs/proxy_policy_unittest.cc
@@ -133,11 +133,11 @@ TEST_F(ProxyPolicyTest, OverridesCommandLineOptions) {
policy.Set(key::kProxyMode, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,
POLICY_SOURCE_CLOUD, std::move(mode_name), nullptr);
policy.Set(key::kProxyBypassList, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,
- POLICY_SOURCE_CLOUD,
- base::WrapUnique(new base::StringValue("abc")), nullptr);
+ POLICY_SOURCE_CLOUD, base::MakeUnique<base::StringValue>("abc"),
+ nullptr);
policy.Set(key::kProxyServer, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,
- POLICY_SOURCE_CLOUD,
- base::WrapUnique(new base::StringValue("ghi")), nullptr);
+ POLICY_SOURCE_CLOUD, base::MakeUnique<base::StringValue>("ghi"),
+ nullptr);
provider_.UpdateChromePolicy(policy);
// First verify that command-line options are set correctly when

Powered by Google App Engine
This is Rietveld 408576698