Change more base::ListValue methods to use std::unique_ptr.

chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc

Index: chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc b/chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc
index 167ef934b4e3c984771751dab317707aa54bdf43..f123af5f22b04b6133fd9bece597016c67d8c474 100644
--- a/chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc
+++ b/chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc
@@ -12,6 +12,7 @@
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/json/json_reader.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
@@ -166,10 +167,9 @@ void AccountsOptionsHandler::HandleUpdateWhitelist(
 
   const user_manager::UserList& users =
       user_manager::UserManager::Get()->GetUsers();
-  for (user_manager::UserList::const_iterator it = users.begin();
-       it < users.end();
-       ++it)
-    new_list->AppendIfNotPresent(new base::StringValue((*it)->email()));
+  for (const auto& user : users)

[danakj, 2016/09/14 01:55:59]

{}

also this is a pointer so const auto*?

[dcheng, 2016/09/14 03:06:11]

Done, I assumed it a std::unique_ptr but I guess I should have just checked!

+    new_list->AppendIfNotPresent(
+        base::MakeUnique<base::StringValue>(user->email()));
 
   if (OwnerSettingsServiceChromeOS* service =
           OwnerSettingsServiceChromeOS::FromWebUI(web_ui())) {