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

Unified Diff: components/user_manager/user_manager_base.cc

Issue 2392693002: Rewrite simple uses of base::ListValue::Append(base::Value*) on CrOS. (Closed)
Patch Set: MakeUnique Created 4 years, 2 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
« no previous file with comments | « components/user_manager/known_user.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/user_manager/user_manager_base.cc
diff --git a/components/user_manager/user_manager_base.cc b/components/user_manager/user_manager_base.cc
index 8cd5d50b9c150a4d705c8c92d5ce25fb12fdb9e8..5c9604083030ab880c09e971a6494611894e3fe0 100644
--- a/components/user_manager/user_manager_base.cc
+++ b/components/user_manager/user_manager_base.cc
@@ -966,7 +966,7 @@ User* UserManagerBase::RemoveRegularOrSupervisedUserFromList(
} else {
if ((*it)->HasGaiaAccount() || (*it)->IsSupervised()) {
const std::string user_email = (*it)->email();
- prefs_users_update->Append(new base::StringValue(user_email));
+ prefs_users_update->AppendString(user_email);
}
++it;
}
« no previous file with comments | « components/user_manager/known_user.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698