Index: ios/chrome/browser/ui/webui/sync_internals/sync_internals_message_handler.cc |
diff --git a/ios/chrome/browser/ui/webui/sync_internals/sync_internals_message_handler.cc b/ios/chrome/browser/ui/webui/sync_internals/sync_internals_message_handler.cc |
index 4a136416c31d2eb21477d4f99ac8c7da66751e4a..942459bc8a7ee839cc6ea952f652e29fdd4c153f 100644 |
--- a/ios/chrome/browser/ui/webui/sync_internals/sync_internals_message_handler.cc |
+++ b/ios/chrome/browser/ui/webui/sync_internals/sync_internals_message_handler.cc |
@@ -7,6 +7,7 @@ |
#include <vector> |
#include "base/logging.h" |
+#include "base/memory/ptr_util.h" |
#include "base/values.h" |
#include "components/browser_sync/profile_sync_service.h" |
#include "components/signin/core/browser/signin_manager.h" |
@@ -125,7 +126,8 @@ void SyncInternalsMessageHandler::HandleRequestListOfTypes( |
ModelTypeSet protocol_types = syncer::ProtocolTypes(); |
for (ModelTypeSet::Iterator it = protocol_types.First(); it.Good(); |
it.Inc()) { |
- type_list->Append(new base::Value(ModelTypeToString(it.Get()))); |
+ type_list->Append( |
+ base::MakeUnique<base::Value>(ModelTypeToString(it.Get()))); |
jdoerrie
2017/04/12 12:37:46
Consider using AppendString and removing the added
vabr (Chromium)
2017/04/12 12:55:04
Done.
|
} |
event_details.Set(syncer::sync_ui_util::kTypes, type_list.release()); |
web_ui()->CallJavascriptFunction( |