jni_generator: Pass object parameters as JavaParamRef.

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 118 matching lines @@
 PrefService* GetPrefService() {
   return GetOriginalProfile()->GetPrefs();
 }
 
 }  // namespace
 
 // ----------------------------------------------------------------------------
 // Native JNI methods
 // ----------------------------------------------------------------------------
 
-static jboolean IsContentSettingManaged(JNIEnv* env, jobject obj,
-    int content_settings_type) {
+static jboolean IsContentSettingManaged(JNIEnv* env,
+                                        const JavaParamRef<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) {
+static jboolean IsContentSettingEnabled(JNIEnv* env,
+                                        const JavaParamRef<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 ||
          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 SetContentSettingEnabled(JNIEnv* env, jobject obj,
-    int content_settings_type, jboolean allow) {
+static void SetContentSettingEnabled(JNIEnv* env,
+                                     const JavaParamRef<jobject>& obj,
+                                     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) {
+static void SetContentSettingForPattern(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj,
+                                        int content_settings_type,
+                                        const JavaParamRef<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) {
+static void GetContentSettingsExceptions(JNIEnv* env,
+                                         const JavaParamRef<jobject>& obj,
+                                         int content_settings_type,
+                                         const JavaParamRef<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) {
+static jboolean GetAcceptCookiesEnabled(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_COOKIES);
 }
 
-static jboolean GetAcceptCookiesManaged(JNIEnv* env, jobject obj) {
+static jboolean GetAcceptCookiesManaged(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return IsContentSettingManaged(CONTENT_SETTINGS_TYPE_COOKIES);
 }
 
-static jboolean GetBlockThirdPartyCookiesEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetBlockThirdPartyCookiesEnabled(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kBlockThirdPartyCookies);
 }
 
-static jboolean GetBlockThirdPartyCookiesManaged(JNIEnv* env, jobject obj) {
+static jboolean GetBlockThirdPartyCookiesManaged(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(prefs::kBlockThirdPartyCookies);
 }
 
-static jboolean GetRememberPasswordsEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetRememberPasswordsEnabled(JNIEnv* env,
+                                            const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(
       password_manager::prefs::kPasswordManagerSavingEnabled);
 }
 
-static jboolean GetPasswordManagerAutoSigninEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetPasswordManagerAutoSigninEnabled(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(
       password_manager::prefs::kPasswordManagerAutoSignin);
 }
 
-static jboolean GetRememberPasswordsManaged(JNIEnv* env, jobject obj) {
+static jboolean GetRememberPasswordsManaged(JNIEnv* env,
+                                            const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(
       password_manager::prefs::kPasswordManagerSavingEnabled);
 }
 
-static jboolean GetPasswordManagerAutoSigninManaged(JNIEnv* env, jobject obj) {
+static jboolean GetPasswordManagerAutoSigninManaged(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(
       password_manager::prefs::kPasswordManagerAutoSignin);
 }
 
-static jboolean GetDoNotTrackEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetDoNotTrackEnabled(JNIEnv* env,
+                                     const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kEnableDoNotTrack);
 }
 
-static jint GetNetworkPredictionOptions(JNIEnv* env, jobject obj) {
+static jint GetNetworkPredictionOptions(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetInteger(prefs::kNetworkPredictionOptions);
 }
 
-static jboolean GetNetworkPredictionManaged(JNIEnv* env, jobject obj) {
+static jboolean GetNetworkPredictionManaged(JNIEnv* env,
+                                            const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(
       prefs::kNetworkPredictionOptions);
 }
 
-static jboolean GetPasswordEchoEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetPasswordEchoEnabled(JNIEnv* env,
+                                       const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kWebKitPasswordEchoEnabled);
 }
 
-static jboolean GetPrintingEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetPrintingEnabled(JNIEnv* env,
+                                   const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kPrintingEnabled);
 }
 
-static jboolean GetPrintingManaged(JNIEnv* env, jobject obj) {
+static jboolean GetPrintingManaged(JNIEnv* env,
+                                   const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(prefs::kPrintingEnabled);
 }
 
-static jboolean GetTranslateEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetTranslateEnabled(JNIEnv* env,
+                                    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kEnableTranslate);
 }
 
-static jboolean GetTranslateManaged(JNIEnv* env, jobject obj) {
+static jboolean GetTranslateManaged(JNIEnv* env,
+                                    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(prefs::kEnableTranslate);
 }
 
-static jboolean GetAutoDetectEncodingEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetAutoDetectEncodingEnabled(JNIEnv* env,
+                                             const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kWebKitUsesUniversalDetector);
 }
 
-static jboolean GetSearchSuggestEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetSearchSuggestEnabled(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kSearchSuggestEnabled);
 }
 
-static jboolean GetSearchSuggestManaged(JNIEnv* env, jobject obj) {
+static jboolean GetSearchSuggestManaged(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(prefs::kSearchSuggestEnabled);
 }
 
-static jboolean GetSafeBrowsingExtendedReportingEnabled(JNIEnv* env,
-                                                        jobject obj) {
+static jboolean GetSafeBrowsingExtendedReportingEnabled(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(
       prefs::kSafeBrowsingExtendedReportingEnabled);
 }
 
-static void SetSafeBrowsingExtendedReportingEnabled(JNIEnv* env,
-                                                    jobject obj,
-                                                    jboolean enabled) {
+static void SetSafeBrowsingExtendedReportingEnabled(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj,
+    jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kSafeBrowsingExtendedReportingEnabled,
                                enabled);
 }
 
-static jboolean GetSafeBrowsingExtendedReportingManaged(JNIEnv* env,
-                                                        jobject obj) {
+static jboolean GetSafeBrowsingExtendedReportingManaged(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(
       prefs::kSafeBrowsingExtendedReportingEnabled);
 }
 
-static jboolean GetSafeBrowsingEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetSafeBrowsingEnabled(JNIEnv* env,
+                                       const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kSafeBrowsingEnabled);
 }
 
-static void SetSafeBrowsingEnabled(JNIEnv* env, jobject obj, jboolean enabled) {
+static void SetSafeBrowsingEnabled(JNIEnv* env,
+                                   const JavaParamRef<jobject>& obj,
+                                   jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kSafeBrowsingEnabled, enabled);
 }
 
-static jboolean GetSafeBrowsingManaged(JNIEnv* env, jobject obj) {
+static jboolean GetSafeBrowsingManaged(JNIEnv* env,
+                                       const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(prefs::kSafeBrowsingEnabled);
 }
 
-static jboolean GetProtectedMediaIdentifierEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetProtectedMediaIdentifierEnabled(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetBooleanForContentSetting(
       CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER);
 }
 
-static jboolean GetPushNotificationsEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetPushNotificationsEnabled(JNIEnv* env,
+                                            const JavaParamRef<jobject>& obj) {
   return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
 }
 
-static jboolean GetAllowLocationEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetAllowLocationEnabled(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_GEOLOCATION);
 }
 
-static jboolean GetLocationAllowedByPolicy(JNIEnv* env, jobject obj) {
+static jboolean GetLocationAllowedByPolicy(JNIEnv* env,
+                                           const JavaParamRef<jobject>& obj) {
   if (!IsContentSettingManaged(CONTENT_SETTINGS_TYPE_GEOLOCATION))
     return false;
   HostContentSettingsMap* content_settings =
       GetOriginalProfile()->GetHostContentSettingsMap();
   return content_settings->GetDefaultContentSetting(
              CONTENT_SETTINGS_TYPE_GEOLOCATION, nullptr) ==
          CONTENT_SETTING_ALLOW;
 }
 
-static jboolean GetAllowLocationUserModifiable(JNIEnv* env, jobject obj) {
+static jboolean GetAllowLocationUserModifiable(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return IsContentSettingUserModifiable(CONTENT_SETTINGS_TYPE_GEOLOCATION);
 }
 
-static jboolean GetAllowLocationManagedByCustodian(JNIEnv* env, jobject obj) {
+static jboolean GetAllowLocationManagedByCustodian(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return IsContentSettingManagedByCustodian(CONTENT_SETTINGS_TYPE_GEOLOCATION);
 }
 
-static jboolean GetResolveNavigationErrorEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetResolveNavigationErrorEnabled(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kAlternateErrorPagesEnabled);
 }
 
-static jboolean GetResolveNavigationErrorManaged(JNIEnv* env, jobject obj) {
+static jboolean GetResolveNavigationErrorManaged(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(
       prefs::kAlternateErrorPagesEnabled);
 }
 
-static jboolean GetCrashReportManaged(JNIEnv* env, jobject obj) {
+static jboolean GetCrashReportManaged(JNIEnv* env,
+                                      const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(
       prefs::kCrashReportingEnabled);
 }
 
-static jboolean GetForceGoogleSafeSearch(JNIEnv* env, jobject obj) {
+static jboolean GetForceGoogleSafeSearch(JNIEnv* env,
+                                         const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kForceGoogleSafeSearch);
 }
 
-static jint GetDefaultSupervisedUserFilteringBehavior(JNIEnv* env,
-                                                      jobject obj) {
+static jint GetDefaultSupervisedUserFilteringBehavior(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetInteger(
       prefs::kDefaultSupervisedUserFilteringBehavior);
 }
 
-static jboolean GetIncognitoModeEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetIncognitoModeEnabled(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   PrefService* prefs = GetPrefService();
   IncognitoModePrefs::Availability incognito_pref =
       IncognitoModePrefs::GetAvailability(prefs);
   DCHECK(incognito_pref == IncognitoModePrefs::ENABLED ||
          incognito_pref == IncognitoModePrefs::DISABLED) <<
              "Unsupported incognito mode preference: " << incognito_pref;
   return incognito_pref != IncognitoModePrefs::DISABLED;
 }
 
-static jboolean GetIncognitoModeManaged(JNIEnv* env, jobject obj) {
+static jboolean GetIncognitoModeManaged(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(
       prefs::kIncognitoModeAvailability);
 }
 
-static jboolean GetFullscreenManaged(JNIEnv* env, jobject obj) {
+static jboolean GetFullscreenManaged(JNIEnv* env,
+                                     const JavaParamRef<jobject>& obj) {
   return IsContentSettingManaged(CONTENT_SETTINGS_TYPE_FULLSCREEN);
 }
 
-static jboolean GetFullscreenAllowed(JNIEnv* env, jobject obj) {
+static jboolean GetFullscreenAllowed(JNIEnv* env,
+                                     const JavaParamRef<jobject>& obj) {
   HostContentSettingsMap* content_settings =
       GetOriginalProfile()->GetHostContentSettingsMap();
   return content_settings->GetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_FULLSCREEN, NULL) == CONTENT_SETTING_ALLOW;
 }
 
-static jboolean GetMetricsReportingEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetMetricsReportingEnabled(JNIEnv* env,
+                                           const JavaParamRef<jobject>& obj) {
   PrefService* local_state = g_browser_process->local_state();
   return local_state->GetBoolean(prefs::kMetricsReportingEnabled);
 }
 
 static void SetMetricsReportingEnabled(JNIEnv* env,
-                                       jobject obj,
+                                       const JavaParamRef<jobject>& obj,
                                        jboolean enabled) {
   PrefService* local_state = g_browser_process->local_state();
   local_state->SetBoolean(prefs::kMetricsReportingEnabled, enabled);
 }
 
-static jboolean HasSetMetricsReporting(JNIEnv* env, jobject obj) {
+static jboolean HasSetMetricsReporting(JNIEnv* env,
+                                       const JavaParamRef<jobject>& obj) {
   PrefService* local_state = g_browser_process->local_state();
   return local_state->HasPrefPath(prefs::kMetricsReportingEnabled);
 }
 
 namespace {
 
 // Redirects a BrowsingDataRemover completion callback back into Java.
 class ClearBrowsingDataObserver : public BrowsingDataRemover::Observer {
  public:
   // |obj| is expected to be the object passed into ClearBrowsingData(); e.g. a
@@ skipping 14 matching lines @@
 
     Java_PrefServiceBridge_browsingDataCleared(
         env, weak_chrome_native_preferences_.get(env).obj());
   }
 
  private:
   JavaObjectWeakGlobalRef weak_chrome_native_preferences_;
 };
 }  // namespace
 
-static void ClearBrowsingData(JNIEnv* env, jobject obj, jboolean history,
-    jboolean cache, jboolean cookies_and_site_data, jboolean passwords,
-    jboolean form_data) {
+static void ClearBrowsingData(JNIEnv* env,
+                              const JavaParamRef<jobject>& obj,
+                              jboolean history,
+                              jboolean cache,
+                              jboolean cookies_and_site_data,
+                              jboolean passwords,
+                              jboolean form_data) {
   // BrowsingDataRemover deletes itself.
   BrowsingDataRemover* browsing_data_remover =
       BrowsingDataRemover::CreateForPeriod(
           GetOriginalProfile(),
           static_cast<BrowsingDataRemover::TimePeriod>(
               BrowsingDataRemover::EVERYTHING));
   browsing_data_remover->AddObserver(new ClearBrowsingDataObserver(env, obj));
 
   int remove_mask = 0;
   if (history)
     remove_mask |= BrowsingDataRemover::REMOVE_HISTORY;
   if (cache)
     remove_mask |= BrowsingDataRemover::REMOVE_CACHE;
   if (cookies_and_site_data) {
     remove_mask |= BrowsingDataRemover::REMOVE_COOKIES;
     remove_mask |= BrowsingDataRemover::REMOVE_SITE_DATA;
   }
   if (passwords)
     remove_mask |= BrowsingDataRemover::REMOVE_PASSWORDS;
   if (form_data)
     remove_mask |= BrowsingDataRemover::REMOVE_FORM_DATA;
   browsing_data_remover->Remove(remove_mask,
                                 BrowsingDataHelper::UNPROTECTED_WEB);
 }
 
-static jboolean CanDeleteBrowsingHistory(JNIEnv* env, jobject obj) {
+static jboolean CanDeleteBrowsingHistory(JNIEnv* env,
+                                         const JavaParamRef<jobject>& obj) {
   return GetPrefService()->GetBoolean(prefs::kAllowDeletingBrowserHistory);
 }
 
-static void SetAllowCookiesEnabled(JNIEnv* env, jobject obj, jboolean allow) {
+static void SetAllowCookiesEnabled(JNIEnv* env,
+                                   const JavaParamRef<jobject>& obj,
+                                   jboolean allow) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_COOKIES,
       allow ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK);
 }
 
-static void SetBlockThirdPartyCookiesEnabled(JNIEnv* env, jobject obj,
+static void SetBlockThirdPartyCookiesEnabled(JNIEnv* env,
+                                             const JavaParamRef<jobject>& obj,
                                              jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kBlockThirdPartyCookies, enabled);
 }
 
-static void SetRememberPasswordsEnabled(JNIEnv* env, jobject obj,
+static void SetRememberPasswordsEnabled(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj,
                                         jboolean allow) {
   GetPrefService()->SetBoolean(
       password_manager::prefs::kPasswordManagerSavingEnabled, allow);
 }
 
-static void SetPasswordManagerAutoSigninEnabled(JNIEnv* env,
-                                                jobject obj,
-                                                jboolean enabled) {
+static void SetPasswordManagerAutoSigninEnabled(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj,
+    jboolean enabled) {
   GetPrefService()->SetBoolean(
       password_manager::prefs::kPasswordManagerAutoSignin, enabled);
 }
 
 static void SetProtectedMediaIdentifierEnabled(JNIEnv* env,
-                                               jobject obj,
+                                               const JavaParamRef<jobject>& obj,
                                                jboolean is_enabled) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER,
       is_enabled ? CONTENT_SETTING_ASK : CONTENT_SETTING_BLOCK);
 }
 
 static void SetAllowLocationEnabled(JNIEnv* env,
-                                    jobject obj,
+                                    const JavaParamRef<jobject>& obj,
                                     jboolean is_enabled) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_GEOLOCATION,
       is_enabled ? CONTENT_SETTING_ASK : CONTENT_SETTING_BLOCK);
 }
 
-static void SetCameraEnabled(JNIEnv* env, jobject obj, jboolean allow) {
+static void SetCameraEnabled(JNIEnv* env,
+                             const JavaParamRef<jobject>& obj,
+                             jboolean allow) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA,
       allow ? CONTENT_SETTING_ASK : CONTENT_SETTING_BLOCK);
 }
 
-static void SetMicEnabled(JNIEnv* env, jobject obj, jboolean allow) {
+static void SetMicEnabled(JNIEnv* env,
+                          const JavaParamRef<jobject>& obj,
+                          jboolean allow) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC,
       allow ? CONTENT_SETTING_ASK : CONTENT_SETTING_BLOCK);
 }
 
-static void SetFullscreenAllowed(JNIEnv* env, jobject obj, jboolean allow) {
+static void SetFullscreenAllowed(JNIEnv* env,
+                                 const JavaParamRef<jobject>& obj,
+                                 jboolean allow) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_FULLSCREEN,
       allow ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_ASK);
 }
 
 static void SetPushNotificationsEnabled(JNIEnv* env,
-                                        jobject obj,
+                                        const JavaParamRef<jobject>& obj,
                                         jboolean allow) {
   HostContentSettingsMap* host_content_settings_map =
       GetOriginalProfile()->GetHostContentSettingsMap();
   host_content_settings_map->SetDefaultContentSetting(
       CONTENT_SETTINGS_TYPE_NOTIFICATIONS,
       allow ? CONTENT_SETTING_ASK : CONTENT_SETTING_BLOCK);
 }
 
-static void SetCrashReporting(JNIEnv* env, jobject obj, jboolean reporting) {
+static void SetCrashReporting(JNIEnv* env,
+                              const JavaParamRef<jobject>& obj,
+                              jboolean reporting) {
   PrefService* local_state = g_browser_process->local_state();
   local_state->SetBoolean(prefs::kCrashReportingEnabled, reporting);
 }
 
-static jboolean CanPredictNetworkActions(JNIEnv* env, jobject obj) {
+static jboolean CanPredictNetworkActions(JNIEnv* env,
+                                         const JavaParamRef<jobject>& obj) {
   return chrome_browser_net::CanPrefetchAndPrerenderUI(GetPrefService());
 }
 
-static void SetDoNotTrackEnabled(JNIEnv* env, jobject obj, jboolean allow) {
+static void SetDoNotTrackEnabled(JNIEnv* env,
+                                 const JavaParamRef<jobject>& obj,
+                                 jboolean allow) {
   GetPrefService()->SetBoolean(prefs::kEnableDoNotTrack, allow);
 }
 
-static ScopedJavaLocalRef<jstring> GetSyncLastAccountName(JNIEnv* env,
-                                                          jobject obj) {
+static ScopedJavaLocalRef<jstring> GetSyncLastAccountName(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env, GetPrefService()->GetString(prefs::kGoogleServicesLastUsername));
 }
 
-static void SetTranslateEnabled(JNIEnv* env, jobject obj, jboolean enabled) {
+static void SetTranslateEnabled(JNIEnv* env,
+                                const JavaParamRef<jobject>& obj,
+                                jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kEnableTranslate, enabled);
 }
 
-static void SetAutoDetectEncodingEnabled(JNIEnv* env, jobject obj,
+static void SetAutoDetectEncodingEnabled(JNIEnv* env,
+                                         const JavaParamRef<jobject>& obj,
                                          jboolean enabled) {
   content::RecordAction(base::UserMetricsAction("AutoDetectChange"));
   GetPrefService()->SetBoolean(prefs::kWebKitUsesUniversalDetector, enabled);
 }
 
-static void ResetTranslateDefaults(JNIEnv* env, jobject obj) {
+static void ResetTranslateDefaults(JNIEnv* env,
+                                   const JavaParamRef<jobject>& obj) {
   scoped_ptr<translate::TranslatePrefs> translate_prefs =
       ChromeTranslateClient::CreateTranslatePrefs(GetPrefService());
   translate_prefs->ResetToDefaults();
 }
 
-static void MigrateJavascriptPreference(JNIEnv* env, jobject obj) {
+static void MigrateJavascriptPreference(JNIEnv* env,
+                                        const JavaParamRef<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);
   SetContentSettingEnabled(env, obj,
       CONTENT_SETTINGS_TYPE_JAVASCRIPT, javascript_enabled);
   GetPrefService()->ClearPref(prefs::kWebKitJavascriptEnabled);
 }
 
-static void MigrateLocationPreference(JNIEnv* env, jobject obj) {
+static void MigrateLocationPreference(JNIEnv* env,
+                                      const JavaParamRef<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);
   DCHECK(retval);
   // Do a restrictive migration. GetAllowLocationEnabled could be
   // non-usermodifiable and we don't want to migrate that.
   if (!location_enabled)
     SetAllowLocationEnabled(env, obj, false);
   GetPrefService()->ClearPref(prefs::kGeolocationEnabled);
 }
 
-static void MigrateProtectedMediaPreference(JNIEnv* env, jobject obj) {
+static void MigrateProtectedMediaPreference(JNIEnv* env,
+                                            const JavaParamRef<jobject>& obj) {
   const PrefService::Preference* pref =
       GetPrefService()->FindPreference(prefs::kProtectedMediaIdentifierEnabled);
   if (!pref || !pref->HasUserSetting())
     return;
   bool pmi_enabled = false;
   bool retval = pref->GetValue()->GetAsBoolean(&pmi_enabled);
   DCHECK(retval);
   // Do a restrictive migration if values disagree.
   if (!pmi_enabled)
     SetProtectedMediaIdentifierEnabled(env, obj, false);
   GetPrefService()->ClearPref(prefs::kProtectedMediaIdentifierEnabled);
 }
 
 static void SetPasswordEchoEnabled(JNIEnv* env,
-                                   jobject obj,
+                                   const JavaParamRef<jobject>& obj,
                                    jboolean passwordEchoEnabled) {
   GetPrefService()->SetBoolean(prefs::kWebKitPasswordEchoEnabled,
                                passwordEchoEnabled);
 }
 
-static jboolean GetCameraEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetCameraEnabled(JNIEnv* env,
+                                 const JavaParamRef<jobject>& obj) {
   return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
 }
 
-static jboolean GetCameraUserModifiable(JNIEnv* env, jobject obj) {
+static jboolean GetCameraUserModifiable(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return IsContentSettingUserModifiable(
              CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
 }
 
-static jboolean GetCameraManagedByCustodian(JNIEnv* env, jobject obj) {
+static jboolean GetCameraManagedByCustodian(JNIEnv* env,
+                                            const JavaParamRef<jobject>& obj) {
   return IsContentSettingManagedByCustodian(
              CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
 }
 
-static jboolean GetMicEnabled(JNIEnv* env, jobject obj) {
+static jboolean GetMicEnabled(JNIEnv* env, const JavaParamRef<jobject>& obj) {
   return GetBooleanForContentSetting(CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC);
 }
 
-static jboolean GetMicUserModifiable(JNIEnv* env, jobject obj) {
+static jboolean GetMicUserModifiable(JNIEnv* env,
+                                     const JavaParamRef<jobject>& obj) {
   return IsContentSettingUserModifiable(
              CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC);
 }
 
-static jboolean GetMicManagedByCustodian(JNIEnv* env, jobject obj) {
+static jboolean GetMicManagedByCustodian(JNIEnv* env,
+                                         const JavaParamRef<jobject>& obj) {
   return IsContentSettingManagedByCustodian(
              CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC);
 }
 
-static void SetJavaScriptAllowed(JNIEnv* env, jobject obj, jstring pattern,
+static void SetJavaScriptAllowed(JNIEnv* env,
+                                 const JavaParamRef<jobject>& obj,
+                                 const JavaParamRef<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_JAVASCRIPT,
       "",
       static_cast<ContentSetting>(setting));
 }
 
-static void SetPopupException(JNIEnv* env, jobject obj, jstring pattern,
+static void SetPopupException(JNIEnv* env,
+                              const JavaParamRef<jobject>& obj,
+                              const JavaParamRef<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 SetSearchSuggestEnabled(JNIEnv* env, jobject obj,
+static void SetSearchSuggestEnabled(JNIEnv* env,
+                                    const JavaParamRef<jobject>& obj,
                                     jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kSearchSuggestEnabled, enabled);
 }
 
-static ScopedJavaLocalRef<jstring> GetContextualSearchPreference(JNIEnv* env,
-                                                                 jobject obj) {
+static ScopedJavaLocalRef<jstring> GetContextualSearchPreference(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env, GetPrefService()->GetString(prefs::kContextualSearchEnabled));
 }
 
-static jboolean GetContextualSearchPreferenceIsManaged(JNIEnv* env,
-                                                       jobject obj) {
+static jboolean GetContextualSearchPreferenceIsManaged(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return GetPrefService()->IsManagedPreference(prefs::kContextualSearchEnabled);
 }
 
-static void SetContextualSearchPreference(JNIEnv* env, jobject obj,
-                                          jstring pref) {
+static void SetContextualSearchPreference(JNIEnv* env,
+                                          const JavaParamRef<jobject>& obj,
+                                          const JavaParamRef<jstring>& pref) {
   GetPrefService()->SetString(prefs::kContextualSearchEnabled,
       ConvertJavaStringToUTF8(env, pref));
 }
 
-static void SetNetworkPredictionOptions(JNIEnv* env, jobject obj, int option) {
+static void SetNetworkPredictionOptions(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj,
+                                        int option) {
   GetPrefService()->SetInteger(prefs::kNetworkPredictionOptions, option);
 }
 
-static void SetResolveNavigationErrorEnabled(JNIEnv* env, jobject obj,
+static void SetResolveNavigationErrorEnabled(JNIEnv* env,
+                                             const JavaParamRef<jobject>& obj,
                                              jboolean enabled) {
   GetPrefService()->SetBoolean(prefs::kAlternateErrorPagesEnabled, enabled);
 }
 
-static jboolean GetFirstRunEulaAccepted(JNIEnv* env, jobject obj) {
+static jboolean GetFirstRunEulaAccepted(JNIEnv* env,
+                                        const JavaParamRef<jobject>& obj) {
   return g_browser_process->local_state()->GetBoolean(prefs::kEulaAccepted);
 }
 
-static void SetEulaAccepted(JNIEnv* env, jobject obj) {
+static void SetEulaAccepted(JNIEnv* env, const JavaParamRef<jobject>& obj) {
   g_browser_process->local_state()->SetBoolean(prefs::kEulaAccepted, true);
 }
 
 static void ResetAcceptLanguages(JNIEnv* env,
-                                 jobject obj,
-                                 jstring default_locale) {
+                                 const JavaParamRef<jobject>& obj,
+                                 const JavaParamRef<jstring>& default_locale) {
   std::string accept_languages(l10n_util::GetStringUTF8(IDS_ACCEPT_LANGUAGES));
   std::string locale_string(ConvertJavaStringToUTF8(env, default_locale));
 
   PrefServiceBridge::PrependToAcceptLanguagesIfNecessary(locale_string,
                                                          &accept_languages);
   GetPrefService()->SetString(prefs::kAcceptLanguages, accept_languages);
 }
 
 // Sends all information about the different versions to Java.
 // From browser_about_handler.cc
-static ScopedJavaLocalRef<jobject> GetAboutVersionStrings(JNIEnv* env,
-                                                          jobject obj) {
+static ScopedJavaLocalRef<jobject> GetAboutVersionStrings(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   std::string os_version = version_info::GetOSType();
   os_version += " " + AndroidAboutAppInfo::GetOsInfo();
 
   base::android::BuildInfo* android_build_info =
         base::android::BuildInfo::GetInstance();
   std::string application(android_build_info->package_label());
   application.append(" ");
   application.append(version_info::GetVersionNumber());
 
   return Java_PrefServiceBridge_createAboutVersionStrings(
       env, ConvertUTF8ToJavaString(env, application).obj(),
       ConvertUTF8ToJavaString(env, os_version).obj());
 }
 
-static ScopedJavaLocalRef<jstring> GetSupervisedUserCustodianName(JNIEnv* env,
-                                                                  jobject obj) {
+static ScopedJavaLocalRef<jstring> GetSupervisedUserCustodianName(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env, GetPrefService()->GetString(prefs::kSupervisedUserCustodianName));
 }
 
 static ScopedJavaLocalRef<jstring> GetSupervisedUserCustodianEmail(
     JNIEnv* env,
-    jobject obj) {
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env, GetPrefService()->GetString(prefs::kSupervisedUserCustodianEmail));
 }
 
 static ScopedJavaLocalRef<jstring> GetSupervisedUserCustodianProfileImageURL(
     JNIEnv* env,
-    jobject obj) {
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env, GetPrefService()->GetString(
                prefs::kSupervisedUserCustodianProfileImageURL));
 }
 
 static ScopedJavaLocalRef<jstring> GetSupervisedUserSecondCustodianName(
     JNIEnv* env,
-    jobject obj) {
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env,
       GetPrefService()->GetString(prefs::kSupervisedUserSecondCustodianName));
 }
 
 static ScopedJavaLocalRef<jstring> GetSupervisedUserSecondCustodianEmail(
     JNIEnv* env,
-    jobject obj) {
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env,
       GetPrefService()->GetString(prefs::kSupervisedUserSecondCustodianEmail));
 }
 
 static ScopedJavaLocalRef<jstring>
-GetSupervisedUserSecondCustodianProfileImageURL(JNIEnv* env, jobject obj) {
+GetSupervisedUserSecondCustodianProfileImageURL(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj) {
   return ConvertUTF8ToJavaString(
       env, GetPrefService()->GetString(
                prefs::kSupervisedUserSecondCustodianProfileImageURL));
 }
 
 // static
 bool PrefServiceBridge::RegisterPrefServiceBridge(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
@@ skipping 38 matching lines @@
     ContentSettingsType content_type) {
   JNIEnv* env = AttachCurrentThread();
   base::android::ScopedJavaLocalRef<jstring> android_permission =
       Java_PrefServiceBridge_getAndroidPermissionForContentSetting(
           env, content_type);
   if (android_permission.is_null())
     return std::string();
 
   return ConvertJavaStringToUTF8(android_permission);
 }