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

Unified Diff: chrome/browser/printing/cloud_print/cloud_print_proxy_service_unittest.cc

Issue 2664753002: Remove base::StringValue (Closed)
Patch Set: Rebase 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/printing/cloud_print/cloud_print_proxy_service_unittest.cc
diff --git a/chrome/browser/printing/cloud_print/cloud_print_proxy_service_unittest.cc b/chrome/browser/printing/cloud_print/cloud_print_proxy_service_unittest.cc
index a7f92acbf050c733fbaaddd3e02ff59b75b5df06..8636b75d1eca75f2bd8e2917a21ce02b9fc0d7c1 100644
--- a/chrome/browser/printing/cloud_print/cloud_print_proxy_service_unittest.cc
+++ b/chrome/browser/printing/cloud_print/cloud_print_proxy_service_unittest.cc
@@ -246,7 +246,7 @@ TEST_F(CloudPrintProxyPolicyTest, StartWithNoPolicyProxyDisabled) {
profile_.GetTestingPrefService();
prefs->SetUserPref(
prefs::kCloudPrintEmail,
- new base::StringValue(MockServiceProcessControl::EnabledUserId()));
+ new base::Value(MockServiceProcessControl::EnabledUserId()));
service.Initialize();
@@ -261,8 +261,7 @@ TEST_F(CloudPrintProxyPolicyTest, StartWithNoPolicyProxyEnabled) {
sync_preferences::TestingPrefServiceSyncable* prefs =
profile_.GetTestingPrefService();
- prefs->SetUserPref(prefs::kCloudPrintEmail,
- new base::StringValue(std::string()));
+ prefs->SetUserPref(prefs::kCloudPrintEmail, new base::Value(std::string()));
service.Initialize();
service.RefreshStatusFromService();
@@ -279,8 +278,7 @@ TEST_F(CloudPrintProxyPolicyTest, StartWithPolicySetProxyDisabled) {
sync_preferences::TestingPrefServiceSyncable* prefs =
profile_.GetTestingPrefService();
- prefs->SetUserPref(prefs::kCloudPrintEmail,
- new base::StringValue(std::string()));
+ prefs->SetUserPref(prefs::kCloudPrintEmail, new base::Value(std::string()));
prefs->SetManagedPref(prefs::kCloudPrintProxyEnabled, new base::Value(false));
service.Initialize();
@@ -297,8 +295,7 @@ TEST_F(CloudPrintProxyPolicyTest, StartWithPolicySetProxyEnabled) {
sync_preferences::TestingPrefServiceSyncable* prefs =
profile_.GetTestingPrefService();
- prefs->SetUserPref(prefs::kCloudPrintEmail,
- new base::StringValue(std::string()));
+ prefs->SetUserPref(prefs::kCloudPrintEmail, new base::Value(std::string()));
prefs->SetManagedPref(prefs::kCloudPrintProxyEnabled, new base::Value(false));
service.Initialize();
@@ -316,7 +313,7 @@ TEST_F(CloudPrintProxyPolicyTest, StartWithNoPolicyProxyDisabledThenSetPolicy) {
profile_.GetTestingPrefService();
prefs->SetUserPref(
prefs::kCloudPrintEmail,
- new base::StringValue(MockServiceProcessControl::EnabledUserId()));
+ new base::Value(MockServiceProcessControl::EnabledUserId()));
service.Initialize();
@@ -335,8 +332,7 @@ TEST_F(CloudPrintProxyPolicyTest, StartWithNoPolicyProxyEnabledThenSetPolicy) {
sync_preferences::TestingPrefServiceSyncable* prefs =
profile_.GetTestingPrefService();
- prefs->SetUserPref(prefs::kCloudPrintEmail,
- new base::StringValue(std::string()));
+ prefs->SetUserPref(prefs::kCloudPrintEmail, new base::Value(std::string()));
service.Initialize();
service.RefreshStatusFromService();
@@ -359,8 +355,7 @@ TEST_F(CloudPrintProxyPolicyTest,
sync_preferences::TestingPrefServiceSyncable* prefs =
profile_.GetTestingPrefService();
- prefs->SetUserPref(prefs::kCloudPrintEmail,
- new base::StringValue(std::string()));
+ prefs->SetUserPref(prefs::kCloudPrintEmail, new base::Value(std::string()));
prefs->SetManagedPref(prefs::kCloudPrintProxyEnabled, new base::Value(false));
service.Initialize();
@@ -380,8 +375,7 @@ TEST_F(CloudPrintProxyPolicyTest,
sync_preferences::TestingPrefServiceSyncable* prefs =
profile_.GetTestingPrefService();
- prefs->SetUserPref(prefs::kCloudPrintEmail,
- new base::StringValue(std::string()));
+ prefs->SetUserPref(prefs::kCloudPrintEmail, new base::Value(std::string()));
prefs->SetManagedPref(prefs::kCloudPrintProxyEnabled, new base::Value(false));
service.Initialize();
@@ -401,7 +395,7 @@ TEST_F(CloudPrintProxyPolicyTest, StartWithNoPolicyProxyDisabledThenEnable) {
profile_.GetTestingPrefService();
prefs->SetUserPref(
prefs::kCloudPrintEmail,
- new base::StringValue(MockServiceProcessControl::EnabledUserId()));
+ new base::Value(MockServiceProcessControl::EnabledUserId()));
service.Initialize();
EXPECT_EQ(std::string(), prefs->GetString(prefs::kCloudPrintEmail));
@@ -423,8 +417,7 @@ TEST_F(CloudPrintProxyPolicyTest,
sync_preferences::TestingPrefServiceSyncable* prefs =
profile_.GetTestingPrefService();
- prefs->SetUserPref(prefs::kCloudPrintEmail,
- new base::StringValue(std::string()));
+ prefs->SetUserPref(prefs::kCloudPrintEmail, new base::Value(std::string()));
prefs->SetManagedPref(prefs::kCloudPrintProxyEnabled, new base::Value(false));
service.Initialize();
« no previous file with comments | « chrome/browser/prefs/session_startup_pref_unittest.cc ('k') | chrome/browser/profiles/profile_window_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698