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

Unified Diff: android_webview/native/aw_contents.cc

Issue 2496183002: Use GetDefaultLocaleListString for returning LocaleList (Closed)
Patch Set: extract setLocale method, since configuration.getLocales() returns the same as LocaleUtils.getDefau… Created 4 years, 1 month 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: android_webview/native/aw_contents.cc
diff --git a/android_webview/native/aw_contents.cc b/android_webview/native/aw_contents.cc
index a8dfed93d68ffff0ee31dfe25a47d422ea1b0091..d305d89fe8d65868cf75bd2ad16f2c3b3080eeb5 100644
--- a/android_webview/native/aw_contents.cc
+++ b/android_webview/native/aw_contents.cc
@@ -107,6 +107,8 @@ bool g_force_auxiliary_bitmap_rendering = false;
std::string g_locale;
+std::string g_locales;
+
const void* kAwContentsUserDataKey = &kAwContentsUserDataKey;
class AwContentsUserData : public base::SupportsUserData::Data {
@@ -149,8 +151,10 @@ AwContents* AwContents::FromID(int render_process_id, int render_view_id) {
// static
void SetLocale(JNIEnv* env,
const JavaParamRef<jclass>&,
- const JavaParamRef<jstring>& locale) {
+ const JavaParamRef<jstring>& locale,
+ const JavaParamRef<jstring>& locales) {
g_locale = ConvertJavaStringToUTF8(env, locale);
+ g_locales = ConvertJavaStringToUTF8(env, locales);
}
// static
@@ -159,6 +163,11 @@ std::string AwContents::GetLocale() {
}
// static
+std::string AwContents::GetLocales() {
+ return g_locales;
+}
+
+// static
AwBrowserPermissionRequestDelegate* AwBrowserPermissionRequestDelegate::FromID(
int render_process_id, int render_frame_id) {
AwContents* aw_contents = AwContents::FromWebContents(

Powered by Google App Engine
This is Rietveld 408576698