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

Unified Diff: chrome/browser/component_updater/supervised_user_whitelist_installer.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/component_updater/supervised_user_whitelist_installer.cc
diff --git a/chrome/browser/component_updater/supervised_user_whitelist_installer.cc b/chrome/browser/component_updater/supervised_user_whitelist_installer.cc
index d0e79cc75ef8cf49232563042cf1ae2ad670a5f1..73e0a94e43e3dfa36f5ae86b4377ab7f79cd9359 100644
--- a/chrome/browser/component_updater/supervised_user_whitelist_installer.cc
+++ b/chrome/browser/component_updater/supervised_user_whitelist_installer.cc
@@ -439,7 +439,7 @@ bool SupervisedUserWhitelistInstallerImpl::UnregisterWhitelistInternal(
base::ListValue* clients = nullptr;
success = whitelist_dict->GetList(kClients, &clients);
- const bool removed = clients->Remove(base::StringValue(client_id), nullptr);
+ const bool removed = clients->Remove(base::Value(client_id), nullptr);
if (!clients->empty())
return removed;
@@ -549,8 +549,8 @@ void SupervisedUserWhitelistInstallerImpl::RegisterWhitelist(
clients = new base::ListValue;
whitelist_dict->Set(kClients, clients);
}
- bool success = clients->AppendIfNotPresent(
- base::MakeUnique<base::StringValue>(client_id));
+ bool success =
+ clients->AppendIfNotPresent(base::MakeUnique<base::Value>(client_id));
DCHECK(success);
}

Powered by Google App Engine
This is Rietveld 408576698