Implement Site Settings \ Images category.

chrome/browser/android/preferences/pref_service_bridge.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/preferences/pref_service_bridge.h"
 
 #include <jni.h>
 
 #include "base/android/build_info.h"
 #include "base/android/jni_android.h"
@@ skipping 134 matching lines @@
 PrefService* GetPrefService() {
   return GetOriginalProfile()->GetPrefs();
 }
 
 }  // namespace
 
 // ----------------------------------------------------------------------------
 // Native JNI methods
 // ----------------------------------------------------------------------------
 
+static jboolean IsContentSettingManaged(JNIEnv* env, jobject obj,
+    int content_settings_type) {
+  return IsContentSettingManaged(
+      static_cast<ContentSettingsType>(content_settings_type));
+}
+
+static jboolean IsContentSettingEnabled(JNIEnv* env, jobject obj,
+    int content_settings_type) {
+  // Before we migrate functions over to this central function, we must verify
+  // that the functionality provided below is correct.
+  DCHECK(content_settings_type == CONTENT_SETTINGS_TYPE_JAVASCRIPT ||

[newt (away), 2015/04/10 18:46:49]

This is a good check to add. Thanks :0

[Finnur, 2015/04/13 12:59:32]

Acknowledged.

+         content_settings_type == CONTENT_SETTINGS_TYPE_IMAGES ||
+         content_settings_type == CONTENT_SETTINGS_TYPE_POPUPS);
+  ContentSettingsType type =
+      static_cast<ContentSettingsType>(content_settings_type);
+  if (type == CONTENT_SETTINGS_TYPE_JAVASCRIPT ||
+      type == CONTENT_SETTINGS_TYPE_POPUPS)
+    return GetBooleanForContentSetting(type);
+
+  HostContentSettingsMap* content_settings =
+      GetOriginalProfile()->GetHostContentSettingsMap();
+  return content_settings->GetDefaultContentSetting(
+      type, nullptr) == CONTENT_SETTING_ALLOW;
+}
+
+static void SetContentSettingDefault(JNIEnv* env, jobject obj,

[newt (away), 2015/04/10 18:46:49]

"SetContentSettingEnabled()"

[Finnur, 2015/04/13 12:59:32]

Done.

+    int content_settings_type, jboolean allow) {
+  // Before we migrate functions over to this central function, we must verify
+  // that the new category supports ALLOW/BLOCK pairs and, if not, handle them.
+  DCHECK(content_settings_type == CONTENT_SETTINGS_TYPE_JAVASCRIPT ||
+         content_settings_type == CONTENT_SETTINGS_TYPE_IMAGES ||
+         content_settings_type == CONTENT_SETTINGS_TYPE_POPUPS);
+  HostContentSettingsMap* host_content_settings_map =
+      GetOriginalProfile()->GetHostContentSettingsMap();
+  host_content_settings_map->SetDefaultContentSetting(
+      static_cast<ContentSettingsType>(content_settings_type),
+      allow ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK);
+}
+
+static void SetContentSettingForPattern(JNIEnv* env, jobject obj,
+    int content_settings_type, jstring pattern, int setting) {
+  HostContentSettingsMap* host_content_settings_map =
+      GetOriginalProfile()->GetHostContentSettingsMap();
+  host_content_settings_map->SetContentSetting(
+      ContentSettingsPattern::FromString(ConvertJavaStringToUTF8(env, pattern)),
+      ContentSettingsPattern::Wildcard(),
+      static_cast<ContentSettingsType>(content_settings_type),
+      "",
+      static_cast<ContentSetting>(setting));
+}
+
+static void GetContentSettingsExceptions(JNIEnv* env, jobject obj,
+    int content_settings_type, jobject list) {
+  HostContentSettingsMap* host_content_settings_map =
+      GetOriginalProfile()->GetHostContentSettingsMap();
+  ContentSettingsForOneType entries;
+  host_content_settings_map->GetSettingsForOneType(
+      static_cast<ContentSettingsType>(content_settings_type), "", &entries);
+  for (size_t i = 0; i < entries.size(); ++i) {
+    Java_PrefServiceBridge_addContentSettingExceptionToList(
+        env, list,
+        content_settings_type,
+        ConvertUTF8ToJavaString(
+            env, entries[i].primary_pattern.ToString()).obj(),
+        ConvertUTF8ToJavaString(
+            env, GetStringForContentSettingsType(entries[i].setting)).obj(),
+        ConvertUTF8ToJavaString(env, entries[i].source).obj());
+  }
+}
+
 static jboolean GetAcceptCookiesEnabled(JNIEnv* env, jobject obj) {
   return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_COOKIES);
 }
 
 static jboolean GetAcceptCookiesManaged(JNIEnv* env, jobject obj) {
   return IsContentSettingManaged(CONTENT_SETTINGS_TYPE_COOKIES);
 }
 
 static jboolean GetBlockThirdPartyCookiesEnabled(JNIEnv* env, jobject obj) {
   return GetPrefService()->GetBoolean(prefs::kBlockThirdPartyCookies);
@@ skipping 294 matching lines @@
 static void SetTranslateEnabled(JNIEnv* env, jobject obj, jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kEnableTranslate, enabled);
 }
 
 static void ResetTranslateDefaults(JNIEnv* env, jobject obj) {
   scoped_ptr<translate::TranslatePrefs> translate_prefs =
       ChromeTranslateClient::CreateTranslatePrefs(GetPrefService());
   translate_prefs->ResetToDefaults();
 }
 
-static jboolean GetJavaScriptManaged(JNIEnv* env, jobject obj) {
-  return IsContentSettingManaged(CONTENT_SETTINGS_TYPE_JAVASCRIPT);
-}
-
-static void SetJavaScriptEnabled(JNIEnv* env, jobject obj, jboolean enabled) {
-  HostContentSettingsMap* host_content_settings_map =
-      GetOriginalProfile()->GetHostContentSettingsMap();
-  host_content_settings_map->SetDefaultContentSetting(
-      CONTENT_SETTINGS_TYPE_JAVASCRIPT,
-      enabled ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK);
-}
-
-static jboolean GetJavaScriptEnabled(JNIEnv* env, jobject obj) {
-  return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_JAVASCRIPT);
-}
-
 static void MigrateJavascriptPreference(JNIEnv* env, jobject obj) {
   const PrefService::Preference* javascript_pref =
       GetPrefService()->FindPreference(prefs::kWebKitJavascriptEnabled);
   DCHECK(javascript_pref);
 
   if (!javascript_pref->HasUserSetting())
     return;
 
   bool javascript_enabled = false;
   bool retval = javascript_pref->GetValue()->GetAsBoolean(&javascript_enabled);
   DCHECK(retval);
-  SetJavaScriptEnabled(env, obj, javascript_enabled);
+  SetContentSettingDefault(env, obj,
+      CONTENT_SETTINGS_TYPE_JAVASCRIPT, javascript_enabled);
   GetPrefService()->ClearPref(prefs::kWebKitJavascriptEnabled);
 }
 
 static void MigrateLocationPreference(JNIEnv* env, jobject obj) {
   const PrefService::Preference* pref =
       GetPrefService()->FindPreference(prefs::kGeolocationEnabled);
   if (!pref || !pref->HasUserSetting())
     return;
   bool location_enabled = false;
   bool retval = pref->GetValue()->GetAsBoolean(&location_enabled);
@@ skipping 19 matching lines @@
   GetPrefService()->ClearPref(prefs::kProtectedMediaIdentifierEnabled);
 }
 
 static void SetPasswordEchoEnabled(JNIEnv* env,
                                    jobject obj,
                                    jboolean passwordEchoEnabled) {
   GetPrefService()->SetBoolean(prefs::kWebKitPasswordEchoEnabled,
                                passwordEchoEnabled);
 }
 
-static jboolean GetAllowPopupsEnabled(JNIEnv* env, jobject obj) {
-  return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_POPUPS);
-}
-
-static jboolean GetAllowPopupsManaged(JNIEnv* env, jobject obj) {
-  return IsContentSettingManaged(CONTENT_SETTINGS_TYPE_POPUPS);
-}
-
-static void SetAllowPopupsEnabled(JNIEnv* env, jobject obj, jboolean allow) {
-  HostContentSettingsMap* host_content_settings_map =
-      GetOriginalProfile()->GetHostContentSettingsMap();
-  host_content_settings_map->SetDefaultContentSetting(
-      CONTENT_SETTINGS_TYPE_POPUPS,
-      allow ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK);
-}
 
 static jboolean GetCameraMicEnabled(JNIEnv* env, jobject obj) {
   return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC) &&
          GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
 }
 
 static jboolean GetCameraMicUserModifiable(JNIEnv* env, jobject obj) {
   return IsContentSettingUserModifiable(
              CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC) &&
          IsContentSettingUserModifiable(
@@ skipping 21 matching lines @@
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetContentSetting(
       ContentSettingsPattern::FromString(ConvertJavaStringToUTF8(env, pattern)),
       ContentSettingsPattern::Wildcard(),
       CONTENT_SETTINGS_TYPE_JAVASCRIPT,
       "",
       static_cast<ContentSetting>(setting));
 }
 
-static void GetJavaScriptExceptions(JNIEnv* env, jobject obj, jobject list) {
-  HostContentSettingsMap* host_content_settings_map =
-      GetOriginalProfile()->GetHostContentSettingsMap();
-  ContentSettingsForOneType entries;
-  host_content_settings_map->GetSettingsForOneType(
-      CONTENT_SETTINGS_TYPE_JAVASCRIPT, "", &entries);
-  for (size_t i = 0; i < entries.size(); ++i) {
-    Java_PrefServiceBridge_addJavaScriptExceptionToList(
-        env, list,
-        ConvertUTF8ToJavaString(
-            env, entries[i].primary_pattern.ToString()).obj(),
-        ConvertUTF8ToJavaString(
-            env, GetStringForContentSettingsType(entries[i].setting)).obj(),
-        ConvertUTF8ToJavaString(env, entries[i].source).obj());
-  }
-}
-
 static void SetPopupException(JNIEnv* env, jobject obj, jstring pattern,
                               int setting) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetContentSetting(
       ContentSettingsPattern::FromString(ConvertJavaStringToUTF8(env, pattern)),
       ContentSettingsPattern::Wildcard(),
       CONTENT_SETTINGS_TYPE_POPUPS,
       "",
       static_cast<ContentSetting>(setting));
 }
 
-static void GetPopupExceptions(JNIEnv* env, jobject obj, jobject list) {
-  HostContentSettingsMap* host_content_settings_map =
-      GetOriginalProfile()->GetHostContentSettingsMap();
-  ContentSettingsForOneType entries;
-  host_content_settings_map->GetSettingsForOneType(
-      CONTENT_SETTINGS_TYPE_POPUPS, "", &entries);
-  for (size_t i = 0; i < entries.size(); ++i) {
-    Java_PrefServiceBridge_insertPopupExceptionToList(
-        env, list,
-        ConvertUTF8ToJavaString(
-            env, entries[i].primary_pattern.ToString()).obj(),
-        ConvertUTF8ToJavaString(
-            env, GetStringForContentSettingsType(entries[i].setting)).obj(),
-        ConvertUTF8ToJavaString(env, entries[i].source).obj());
-  }
-}
-
 static void SetSearchSuggestEnabled(JNIEnv* env, jobject obj,
                                     jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kSearchSuggestEnabled, enabled);
 }
 
 static jstring GetContextualSearchPreference(JNIEnv* env, jobject obj) {
   return ConvertUTF8ToJavaString(
       env, GetPrefService()->GetString(prefs::kContextualSearchEnabled)).
           Release();
 }
@@ skipping 161 matching lines @@
                                                                jobject obj) {
   return ConvertUTF8ToJavaString(
       env,
       GetPrefService()->GetString(
           prefs::kSupervisedUserSecondCustodianProfileImageURL)).Release();
 }
 
 bool RegisterPrefServiceBridge(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }