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

Unified Diff: components/physical_web/webui/physical_web_base_message_handler.cc

Issue 2889163002: Remove raw DictionaryValue::Set in //components (Closed)
Patch Set: Nits Created 3 years, 7 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: components/physical_web/webui/physical_web_base_message_handler.cc
diff --git a/components/physical_web/webui/physical_web_base_message_handler.cc b/components/physical_web/webui/physical_web_base_message_handler.cc
index dd28dc3dd22c89caf8903913d27dfe60e26af510..f68cfb08c74b9ab8b080689d7f0400a77349871f 100644
--- a/components/physical_web/webui/physical_web_base_message_handler.cc
+++ b/components/physical_web/webui/physical_web_base_message_handler.cc
@@ -4,6 +4,8 @@
#include "components/physical_web/webui/physical_web_base_message_handler.h"
+#include <utility>
+
#include "base/bind.h"
#include "base/memory/ptr_util.h"
#include "base/metrics/histogram_macros.h"
@@ -88,7 +90,7 @@ void PhysicalWebBaseMessageHandler::PushNearbyURLs() {
++index;
}
- results.Set(physical_web_ui::kMetadata, metadata.release());
+ results.Set(physical_web_ui::kMetadata, std::move(metadata));
// Pass the list of Physical Web URL metadata to the WebUI. A jstemplate will
// create a list view with an item for each URL.
« no previous file with comments | « components/login/screens/screen_context.cc ('k') | components/policy/core/common/config_dir_policy_loader_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698