Chromium Code Reviews| 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( |