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

Unified Diff: chrome/browser/extensions/api/proxy/proxy_api_helpers_unittest.cc

Issue 2785883003: Use unique_ptr<DictionaryValue> in ProxyConfigDictionary (Closed)
Patch Set: Created 3 years, 9 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/proxy/proxy_api_helpers_unittest.cc
diff --git a/chrome/browser/extensions/api/proxy/proxy_api_helpers_unittest.cc b/chrome/browser/extensions/api/proxy/proxy_api_helpers_unittest.cc
index 6dcf63a05241c0d1b70176eaea8963c8fd025006..261f3e0eaa1262e8dd957b4949460a69e5231294 100644
--- a/chrome/browser/extensions/api/proxy/proxy_api_helpers_unittest.cc
+++ b/chrome/browser/extensions/api/proxy/proxy_api_helpers_unittest.cc
@@ -222,43 +222,43 @@ TEST(ExtensionProxyApiHelpers, GetBypassListFromExtensionPref) {
TEST(ExtensionProxyApiHelpers, CreateProxyConfigDict) {
std::string error;
- std::unique_ptr<base::DictionaryValue> exp_direct(
- ProxyConfigDictionary::CreateDirect());
+ std::unique_ptr<base::DictionaryValue> exp_direct =
+ ProxyConfigDictionary::CreateDirect();
std::unique_ptr<base::DictionaryValue> out_direct(CreateProxyConfigDict(
ProxyPrefs::MODE_DIRECT, false, std::string(), std::string(),
std::string(), std::string(), &error));
EXPECT_TRUE(base::Value::Equals(exp_direct.get(), out_direct.get()));
- std::unique_ptr<base::DictionaryValue> exp_auto(
- ProxyConfigDictionary::CreateAutoDetect());
+ std::unique_ptr<base::DictionaryValue> exp_auto =
+ ProxyConfigDictionary::CreateAutoDetect();
std::unique_ptr<base::DictionaryValue> out_auto(CreateProxyConfigDict(
ProxyPrefs::MODE_AUTO_DETECT, false, std::string(), std::string(),
std::string(), std::string(), &error));
EXPECT_TRUE(base::Value::Equals(exp_auto.get(), out_auto.get()));
- std::unique_ptr<base::DictionaryValue> exp_pac_url(
- ProxyConfigDictionary::CreatePacScript(kSamplePacScriptUrl, false));
+ std::unique_ptr<base::DictionaryValue> exp_pac_url =
+ ProxyConfigDictionary::CreatePacScript(kSamplePacScriptUrl, false);
std::unique_ptr<base::DictionaryValue> out_pac_url(CreateProxyConfigDict(
ProxyPrefs::MODE_PAC_SCRIPT, false, kSamplePacScriptUrl, std::string(),
std::string(), std::string(), &error));
EXPECT_TRUE(base::Value::Equals(exp_pac_url.get(), out_pac_url.get()));
- std::unique_ptr<base::DictionaryValue> exp_pac_data(
- ProxyConfigDictionary::CreatePacScript(kSamplePacScriptAsDataUrl, false));
+ std::unique_ptr<base::DictionaryValue> exp_pac_data =
+ ProxyConfigDictionary::CreatePacScript(kSamplePacScriptAsDataUrl, false);
std::unique_ptr<base::DictionaryValue> out_pac_data(CreateProxyConfigDict(
ProxyPrefs::MODE_PAC_SCRIPT, false, std::string(), kSamplePacScript,
std::string(), std::string(), &error));
EXPECT_TRUE(base::Value::Equals(exp_pac_data.get(), out_pac_data.get()));
- std::unique_ptr<base::DictionaryValue> exp_fixed(
- ProxyConfigDictionary::CreateFixedServers("foo:80", "localhost"));
+ std::unique_ptr<base::DictionaryValue> exp_fixed =
+ ProxyConfigDictionary::CreateFixedServers("foo:80", "localhost");
std::unique_ptr<base::DictionaryValue> out_fixed(CreateProxyConfigDict(
ProxyPrefs::MODE_FIXED_SERVERS, false, std::string(), std::string(),
"foo:80", "localhost", &error));
EXPECT_TRUE(base::Value::Equals(exp_fixed.get(), out_fixed.get()));
- std::unique_ptr<base::DictionaryValue> exp_system(
- ProxyConfigDictionary::CreateSystem());
+ std::unique_ptr<base::DictionaryValue> exp_system =
+ ProxyConfigDictionary::CreateSystem();
std::unique_ptr<base::DictionaryValue> out_system(CreateProxyConfigDict(
ProxyPrefs::MODE_SYSTEM, false, std::string(), std::string(),
std::string(), std::string(), &error));

Powered by Google App Engine
This is Rietveld 408576698