Remove now-unnecessary .obj() in Java method calls.

chrome/browser/android/download/download_manager_service.cc

 // Copyright 2015 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/download/download_manager_service.h"
 
 #include "base/android/jni_string.h"
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 31 matching lines @@
 
 // static
 void DownloadManagerService::OnDownloadCanceled(
     content::DownloadItem* download,
     DownloadController::DownloadCancelReason reason) {
   bool has_no_external_storage =
       (reason == DownloadController::CANCEL_REASON_NO_EXTERNAL_STORAGE);
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jstring> jname =
       ConvertUTF8ToJavaString(env, download->GetURL().ExtractFileName());
-  Java_DownloadManagerService_onDownloadItemCanceled(
-      env, jname.obj(), has_no_external_storage);
+  Java_DownloadManagerService_onDownloadItemCanceled(env, jname,
+                                                     has_no_external_storage);
   DownloadController::RecordDownloadCancelReason(reason);
 }
 
 
 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& jobj) {
   Profile* profile = ProfileManager::GetActiveUserProfile();
   DownloadManagerService* service =
       new DownloadManagerService(env, jobj);
   DownloadService* download_service =
       DownloadServiceFactory::GetForBrowserContext(profile);
@@ skipping 73 matching lines @@
 
   // Search through the downloads for the entry with the given GUID.
   content::DownloadManager::DownloadVector all_items;
   manager->GetAllDownloads(&all_items);
   std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
   for (auto item : all_items) {
     if (item->GetGuid() != download_guid)
       continue;
 
     Java_DownloadManagerService_onDownloadItemUpdated(
-        env,
-        java_ref_.obj(),
-        ConvertUTF8ToJavaString(env, item->GetGuid()).obj(),
-        ConvertUTF8ToJavaString(
-            env, item->GetFileNameToReportUser().value()).obj(),
-        ConvertUTF8ToJavaString(
-            env, item->GetTargetFilePath().value()).obj(),
-        ConvertUTF8ToJavaString(env, item->GetTabUrl().spec()).obj(),
-        ConvertUTF8ToJavaString(env, item->GetMimeType()).obj(),
-        item->GetStartTime().ToJavaTime(),
-        item->GetTotalBytes());
+        env, java_ref_, ConvertUTF8ToJavaString(env, item->GetGuid()),
+        ConvertUTF8ToJavaString(env, item->GetFileNameToReportUser().value()),
+        ConvertUTF8ToJavaString(env, item->GetTargetFilePath().value()),
+        ConvertUTF8ToJavaString(env, item->GetTabUrl().spec()),
+        ConvertUTF8ToJavaString(env, item->GetMimeType()),
+        item->GetStartTime().ToJavaTime(), item->GetTotalBytes());
     break;
   }
 }
 
 void DownloadManagerService::GetAllDownloadsInternal() {
   content::DownloadManager* manager = GetDownloadManager(false);
   if (java_ref_.is_null() || !manager)
     return;
 
   content::DownloadManager::DownloadVector all_items;
   manager->GetAllDownloads(&all_items);
 
   // Create a Java array of all of the visible DownloadItems.
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> j_download_item_list =
-      Java_DownloadManagerService_createDownloadItemList(env, java_ref_.obj());
+      Java_DownloadManagerService_createDownloadItemList(env, java_ref_);
 
   for (size_t i = 0; i < all_items.size(); i++) {
     content::DownloadItem* item = all_items[i];
     if (!ShouldShowDownloadItem(item))
       continue;
 
     Java_DownloadManagerService_addDownloadItemToList(
-        env,
-        java_ref_.obj(),
-        j_download_item_list.obj(),
-        ConvertUTF8ToJavaString(env, item->GetGuid()).obj(),
-        ConvertUTF8ToJavaString(
-            env, item->GetFileNameToReportUser().value()).obj(),
-        ConvertUTF8ToJavaString(
-            env, item->GetTargetFilePath().value()).obj(),
-        ConvertUTF8ToJavaString(env, item->GetTabUrl().spec()).obj(),
-        ConvertUTF8ToJavaString(env, item->GetMimeType()).obj(),
-        item->GetStartTime().ToJavaTime(),
-        item->GetTotalBytes());
+        env, java_ref_, j_download_item_list,
+        ConvertUTF8ToJavaString(env, item->GetGuid()),
+        ConvertUTF8ToJavaString(env, item->GetFileNameToReportUser().value()),
+        ConvertUTF8ToJavaString(env, item->GetTargetFilePath().value()),
+        ConvertUTF8ToJavaString(env, item->GetTabUrl().spec()),
+        ConvertUTF8ToJavaString(env, item->GetMimeType()),
+        item->GetStartTime().ToJavaTime(), item->GetTotalBytes());
   }
 
-  Java_DownloadManagerService_onAllDownloadsRetrieved(
-      env, java_ref_.obj(), j_download_item_list.obj());
+  Java_DownloadManagerService_onAllDownloadsRetrieved(env, java_ref_,
+                                                      j_download_item_list);
 }
 
 
 void DownloadManagerService::CancelDownload(
     JNIEnv* env,
     jobject obj,
     const JavaParamRef<jstring>& jdownload_guid,
     bool is_off_the_record,
     bool is_notification_dismissed) {
   std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
@@ skipping 36 matching lines @@
   // Monitor all DownloadItems for changes.
   content::DownloadManager* manager = GetDownloadManager(false);
   if (manager)
     original_notifier_.reset(new AllDownloadItemNotifier(manager, this));
 }
 
 void DownloadManagerService::OnDownloadRemoved(
     content::DownloadManager* manager, content::DownloadItem* item) {
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_DownloadManagerService_onDownloadItemRemoved(
-      env,
-      java_ref_.obj(),
-      ConvertUTF8ToJavaString(env, item->GetGuid()).obj());
+      env, java_ref_, ConvertUTF8ToJavaString(env, item->GetGuid()));
 }
 
 void DownloadManagerService::ResumeDownloadInternal(
     const std::string& download_guid, bool is_off_the_record) {
   content::DownloadManager* manager = GetDownloadManager(is_off_the_record);
   if (!manager) {
     OnResumptionFailed(download_guid);
     return;
   }
   content::DownloadItem* item = manager->GetDownloadByGuid(download_guid);
@@ skipping 84 matching lines @@
                             base::Unretained(this), download_guid));
   DownloadController::RecordDownloadCancelReason(
       DownloadController::CANCEL_REASON_NOT_CANCELED);
 }
 
 void DownloadManagerService::OnResumptionFailedInternal(
     const std::string& download_guid) {
   if (!java_ref_.is_null()) {
     JNIEnv* env = base::android::AttachCurrentThread();
     Java_DownloadManagerService_onResumptionFailed(
-        env, java_ref_.obj(),
-        ConvertUTF8ToJavaString(env, download_guid).obj());
+        env, java_ref_, ConvertUTF8ToJavaString(env, download_guid));
   }
   if (!resume_callback_for_testing_.is_null())
     resume_callback_for_testing_.Run(false);
 }
 
 content::DownloadManager* DownloadManagerService::GetDownloadManager(
     bool is_off_the_record) {
   Profile* profile = ProfileManager::GetActiveUserProfile();
   if (is_off_the_record)
     profile = profile->GetOffTheRecordProfile();
   return content::BrowserContext::GetDownloadManager(profile);
 }