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

Unified Diff: chrome/browser/android/data_usage/external_data_use_observer_bridge.cc

Issue 2237943002: Remove now-unnecessary .obj() in Java method calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@switch-context
Patch Set: Rebase *again* :( Created 4 years, 4 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/data_usage/external_data_use_observer_bridge.cc
diff --git a/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc b/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc
index d501350b62b65a2ff0bfee3fa3c87d67bf7f2244..b9438e9412e45bf8f5538dbd91d73713b1c067ba 100644
--- a/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc
+++ b/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc
@@ -52,8 +52,7 @@ ExternalDataUseObserverBridge::~ExternalDataUseObserverBridge() {
if (j_external_data_use_observer_.is_null())
return;
JNIEnv* env = base::android::AttachCurrentThread();
- Java_ExternalDataUseObserver_onDestroy(env,
- j_external_data_use_observer_.obj());
+ Java_ExternalDataUseObserver_onDestroy(env, j_external_data_use_observer_);
}
void ExternalDataUseObserverBridge::Init(
@@ -79,8 +78,8 @@ void ExternalDataUseObserverBridge::Init(
DCHECK(!j_external_data_use_observer_.is_null());
Java_ExternalDataUseObserver_initControlAppManager(
- env, j_external_data_use_observer_.obj(),
- ConvertUTF8ToJavaString(env, GetControlAppPackageName()).obj());
+ env, j_external_data_use_observer_,
+ ConvertUTF8ToJavaString(env, GetControlAppPackageName()));
}
void ExternalDataUseObserverBridge::FetchMatchingRules() const {
@@ -88,7 +87,7 @@ void ExternalDataUseObserverBridge::FetchMatchingRules() const {
JNIEnv* env = base::android::AttachCurrentThread();
Java_ExternalDataUseObserver_fetchMatchingRules(
- env, j_external_data_use_observer_.obj());
+ env, j_external_data_use_observer_);
}
void ExternalDataUseObserverBridge::FetchMatchingRulesDone(
@@ -151,10 +150,9 @@ void ExternalDataUseObserverBridge::ReportDataUse(
start_time_milliseconds = end_time_milliseconds - 1;
Java_ExternalDataUseObserver_reportDataUse(
- env, j_external_data_use_observer_.obj(),
- ConvertUTF8ToJavaString(env, label).obj(),
- ConvertUTF8ToJavaString(env, tag).obj(), connection_type,
- ConvertUTF8ToJavaString(env, mcc_mnc).obj(), start_time_milliseconds,
+ env, j_external_data_use_observer_, ConvertUTF8ToJavaString(env, label),
+ ConvertUTF8ToJavaString(env, tag), connection_type,
+ ConvertUTF8ToJavaString(env, mcc_mnc), start_time_milliseconds,
end_time_milliseconds, bytes_downloaded, bytes_uploaded);
}
« no previous file with comments | « chrome/browser/android/cookies/cookies_fetcher.cc ('k') | chrome/browser/android/dom_distiller/distiller_ui_handle_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698