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

Unified Diff: chrome/browser/android/omnibox/autocomplete_controller_android.cc

Issue 1841653003: Drop |languages| from {Format,Elide}Url* and IDNToUnicode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased, most Android targets locally built successfully Created 4 years, 9 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/android/omnibox/autocomplete_controller_android.cc
diff --git a/chrome/browser/android/omnibox/autocomplete_controller_android.cc b/chrome/browser/android/omnibox/autocomplete_controller_android.cc
index 2551d595812009c97dffb204e4d23e3745066b81..c05e9b9500f88ddd08b091ac03c9dedee63ff654 100644
--- a/chrome/browser/android/omnibox/autocomplete_controller_android.cc
+++ b/chrome/browser/android/omnibox/autocomplete_controller_android.cc
@@ -558,15 +558,9 @@ AutocompleteControllerAndroid::BuildOmniboxSuggestion(
base::string16 AutocompleteControllerAndroid::FormatURLUsingAcceptLanguages(
Peter Kasting 2016/04/05 02:42:32 This function name is now inaccurate.
jungshik at Google 2016/04/05 18:56:18 Thanks for catching it. Turned out that nobody ca
GURL url) {
- if (profile_ == NULL)
- return base::string16();
-
- std::string languages(
- profile_->GetPrefs()->GetString(prefs::kAcceptLanguages));
-
return url_formatter::FormatUrl(
- url, languages, url_formatter::kFormatUrlOmitAll,
- net::UnescapeRule::SPACES, nullptr, nullptr, nullptr);
+ url, url_formatter::kFormatUrlOmitAll, net::UnescapeRule::SPACES,
+ nullptr, nullptr, nullptr);
}
ScopedJavaLocalRef<jobject>

Powered by Google App Engine
This is Rietveld 408576698