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

Unified Diff: chrome/browser/ui/webui/chromeos/login/network_screen_handler.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
Index: chrome/browser/ui/webui/chromeos/login/network_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/network_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/network_screen_handler.cc
index 91e369b74260ef075f78527ec452e39163b591f5..e2fa80c8fb4654bd61621841cd2a72ecf84973ab 100644
--- a/chrome/browser/ui/webui/chromeos/login/network_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/network_screen_handler.cc
@@ -6,6 +6,8 @@
#include <stddef.h>
+#include <utility>
+
#include "base/bind.h"
#include "base/bind_helpers.h"
#include "base/command_line.h"
@@ -285,7 +287,7 @@ base::ListValue* NetworkScreenHandler::GetTimezoneList() {
timezone_option->SetString("value", timezone_id);
timezone_option->SetString("title", timezone_name);
timezone_option->SetBoolean("selected", timezone_id == current_timezone_id);
- timezone_list->Append(timezone_option.release());
+ timezone_list->Append(std::move(timezone_option));
}
return timezone_list.release();

Powered by Google App Engine
This is Rietveld 408576698