Stop passing raw pointers to base::Value API in c/b/ui

chrome/browser/ui/webui/translate_internals/translate_internals_handler.cc

Index: chrome/browser/ui/webui/translate_internals/translate_internals_handler.cc
diff --git a/chrome/browser/ui/webui/translate_internals/translate_internals_handler.cc b/chrome/browser/ui/webui/translate_internals/translate_internals_handler.cc
index 8827d4e86ab9362a6db8bab57ccf75bce5c9d198..9f140829058c0c353a89a0eed4ecf377cccae2b1 100644
--- a/chrome/browser/ui/webui/translate_internals/translate_internals_handler.cc
+++ b/chrome/browser/ui/webui/translate_internals/translate_internals_handler.cc
@@ -5,10 +5,12 @@
 #include "chrome/browser/ui/webui/translate_internals/translate_internals_handler.h"
 
 #include <map>
+#include <utility>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/memory/ptr_util.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ -87,41 +89,46 @@ void TranslateInternalsHandler::Observe(
   }
 
   base::DictionaryValue dict;
-  dict.Set("time",
-           new base::Value(language_detection_details->time.ToJsTime()));
-  dict.Set("url", new base::Value(language_detection_details->url.spec()));
+  dict.Set("time", base::MakeUnique<base::Value>(

[jdoerrie, 2017/04/13 09:08:52]

SetDouble

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

+                       language_detection_details->time.ToJsTime()));
+  dict.Set("url", base::MakeUnique<base::Value>(

[jdoerrie, 2017/04/13 09:08:52]

SetString

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

+                      language_detection_details->url.spec()));
   dict.Set("content_language",

[jdoerrie, 2017/04/13 09:08:53]

SetString

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

-           new base::Value(language_detection_details->content_language));
-  dict.Set("cld_language",
-           new base::Value(language_detection_details->cld_language));
-  dict.Set("is_cld_reliable",
-           new base::Value(language_detection_details->is_cld_reliable));
-  dict.Set("has_notranslate",
-           new base::Value(language_detection_details->has_notranslate));
+           base::MakeUnique<base::Value>(
+               language_detection_details->content_language));
+  dict.Set("cld_language", base::MakeUnique<base::Value>(

[jdoerrie, 2017/04/13 09:08:52]

SetString

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

+                               language_detection_details->cld_language));
+  dict.Set("is_cld_reliable", base::MakeUnique<base::Value>(

[jdoerrie, 2017/04/13 09:08:53]

SetBoolean

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

+                                  language_detection_details->is_cld_reliable));
+  dict.Set("has_notranslate", base::MakeUnique<base::Value>(

[jdoerrie, 2017/04/13 09:08:52]

SetBoolean

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

+                                  language_detection_details->has_notranslate));
   dict.Set("html_root_language",

[jdoerrie, 2017/04/13 09:08:53]

SetString

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

-           new base::Value(language_detection_details->html_root_language));
+           base::MakeUnique<base::Value>(
+               language_detection_details->html_root_language));
   dict.Set("adopted_language",

[jdoerrie, 2017/04/13 09:08:52]

SetString

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

-           new base::Value(language_detection_details->adopted_language));
-  dict.Set("content", new base::Value(language_detection_details->contents));
+           base::MakeUnique<base::Value>(
+               language_detection_details->adopted_language));
+  dict.Set("content",

[jdoerrie, 2017/04/13 09:08:53]

SetString

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

+           base::MakeUnique<base::Value>(language_detection_details->contents));
   SendMessageToJs("languageDetectionInfoAdded", dict);
 }
 
 void TranslateInternalsHandler::OnTranslateError(
     const translate::TranslateErrorDetails& details) {
   base::DictionaryValue dict;
-  dict.Set("time", new base::Value(details.time.ToJsTime()));
-  dict.Set("url", new base::Value(details.url.spec()));
-  dict.Set("error", new base::Value(details.error));
+  dict.Set("time", base::MakeUnique<base::Value>(details.time.ToJsTime()));
+  dict.Set("url", base::MakeUnique<base::Value>(details.url.spec()));
+  dict.Set("error", base::MakeUnique<base::Value>(details.error));

[jdoerrie, 2017/04/13 09:08:52]

SetDouble
SetString
SetInteger

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

   SendMessageToJs("translateErrorDetailsAdded", dict);
 }
 
 void TranslateInternalsHandler::OnTranslateEvent(
     const translate::TranslateEventDetails& details) {
   base::DictionaryValue dict;
-  dict.Set("time", new base::Value(details.time.ToJsTime()));
-  dict.Set("filename", new base::Value(details.filename));
-  dict.Set("line", new base::Value(details.line));
-  dict.Set("message", new base::Value(details.message));
+  dict.Set("time", base::MakeUnique<base::Value>(details.time.ToJsTime()));
+  dict.Set("filename", base::MakeUnique<base::Value>(details.filename));
+  dict.Set("line", base::MakeUnique<base::Value>(details.line));
+  dict.Set("message", base::MakeUnique<base::Value>(details.message));

[jdoerrie, 2017/04/13 09:08:52]

SetDouble
SetString
SetInteger
SetString

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

   SendMessageToJs("translateEventDetailsAdded", dict);
 }
 
@@ -210,7 +217,7 @@ void TranslateInternalsHandler::SendPrefsToJs() {
   for (const char* key : keys) {
     const PrefService::Preference* pref = prefs->FindPreference(key);
     if (pref)
-      dict.Set(key, pref->GetValue()->DeepCopy());
+      dict.Set(key, base::MakeUnique<base::Value>(*pref->GetValue()));
   }
 
   SendMessageToJs("prefsUpdated", dict);
@@ -224,8 +231,8 @@ void TranslateInternalsHandler::SendSupportedLanguagesToJs() {
   base::Time last_updated =
       translate::TranslateDownloadManager::GetSupportedLanguagesLastUpdated();
 
-  base::ListValue* languages_list = new base::ListValue();
-  base::ListValue* alpha_languages_list = new base::ListValue();
+  auto languages_list = base::MakeUnique<base::ListValue>();
+  auto alpha_languages_list = base::MakeUnique<base::ListValue>();
   for (std::vector<std::string>::iterator it = languages.begin();
        it != languages.end(); ++it) {
     const std::string& lang = *it;
@@ -234,9 +241,10 @@ void TranslateInternalsHandler::SendSupportedLanguagesToJs() {
       alpha_languages_list->AppendString(lang);
   }
 
-  dict.Set("languages", languages_list);
-  dict.Set("alpha_languages", alpha_languages_list);
-  dict.Set("last_updated", new base::Value(last_updated.ToJsTime()));
+  dict.Set("languages", std::move(languages_list));
+  dict.Set("alpha_languages", std::move(alpha_languages_list));
+  dict.Set("last_updated",

[jdoerrie, 2017/04/13 09:08:52]

SetDouble

[vabr (Chromium), 2017/04/13 12:03:15]

Done.

+           base::MakeUnique<base::Value>(last_updated.ToJsTime()));
   SendMessageToJs("supportedLanguagesUpdated", dict);
 }
 
@@ -249,8 +257,8 @@ void TranslateInternalsHandler::SendCountryToJs(bool was_updated) {
 
   base::DictionaryValue dict;
   if (!country.empty()) {
-    dict.Set("country", new base::Value(country));
-    dict.Set("update", new base::Value(was_updated));
+    dict.Set("country", base::MakeUnique<base::Value>(country));
+    dict.Set("update", base::MakeUnique<base::Value>(was_updated));

[jdoerrie, 2017/04/13 09:08:53]

SetString
SetBoolean

[vabr (Chromium), 2017/04/13 12:03:14]

Done.

   }
   SendMessageToJs("countryUpdated", dict);
 }