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

Unified Diff: chrome/browser/ui/webui/options/language_options_handler_common.cc

Issue 2820823005: Revert of Stop passing raw pointers to base::Value API in c/b/ui (Closed)
Patch Set: Created 3 years, 8 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/options/language_options_handler_common.cc
diff --git a/chrome/browser/ui/webui/options/language_options_handler_common.cc b/chrome/browser/ui/webui/options/language_options_handler_common.cc
index fd127384e773e6139bb9f32a17eb41225ccf62d4..151d4d70b422827517737f4850fc91f20cc55013 100644
--- a/chrome/browser/ui/webui/options/language_options_handler_common.cc
+++ b/chrome/browser/ui/webui/options/language_options_handler_common.cc
@@ -14,7 +14,6 @@
#include "base/bind.h"
#include "base/command_line.h"
#include "base/macros.h"
-#include "base/memory/ptr_util.h"
#include "base/metrics/user_metrics.h"
#include "base/strings/string_util.h"
#include "base/strings/stringprintf.h"
@@ -118,14 +117,13 @@
std::vector<std::string> languages;
translate::TranslateDownloadManager::GetSupportedLanguages(&languages);
- auto languages_list = base::MakeUnique<base::ListValue>();
+ base::ListValue* languages_list = new base::ListValue();
for (std::vector<std::string>::iterator it = languages.begin();
it != languages.end(); ++it) {
languages_list->AppendString(*it);
}
- localized_strings->Set("translateSupportedLanguages",
- std::move(languages_list));
+ localized_strings->Set("translateSupportedLanguages", languages_list);
}
void LanguageOptionsHandlerCommon::Uninitialize() {
@@ -185,9 +183,8 @@
base::Value(language));
}
-std::unique_ptr<base::DictionaryValue>
-LanguageOptionsHandlerCommon::GetUILanguageCodeSet() {
- auto dictionary = base::MakeUnique<base::DictionaryValue>();
+base::DictionaryValue* LanguageOptionsHandlerCommon::GetUILanguageCodeSet() {
+ base::DictionaryValue* dictionary = new base::DictionaryValue();
const std::vector<std::string>& available_locales =
l10n_util::GetAvailableLocales();
for (size_t i = 0; i < available_locales.size(); ++i)
@@ -195,9 +192,9 @@
return dictionary;
}
-std::unique_ptr<base::DictionaryValue>
+base::DictionaryValue*
LanguageOptionsHandlerCommon::GetSpellCheckLanguageCodeSet() {
- auto dictionary = base::MakeUnique<base::DictionaryValue>();
+ base::DictionaryValue* dictionary = new base::DictionaryValue();
std::vector<std::string> spell_check_languages;
spellcheck::SpellCheckLanguages(&spell_check_languages);
for (size_t i = 0; i < spell_check_languages.size(); ++i) {

Powered by Google App Engine
This is Rietveld 408576698