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

Unified Diff: android_webview/native/aw_web_contents_delegate.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: android_webview/native/aw_web_contents_delegate.cc
diff --git a/android_webview/native/aw_web_contents_delegate.cc b/android_webview/native/aw_web_contents_delegate.cc
index a586e44c268be8b8e8ee4f0c111a90b5ca169815..9588586b41954fd8830a0ca5bea50fb1a71da91e 100644
--- a/android_webview/native/aw_web_contents_delegate.cc
+++ b/android_webview/native/aw_web_contents_delegate.cc
@@ -143,8 +143,8 @@ void AwWebContentsDelegate::AddNewContents(WebContents* source,
bool create_popup = false;
if (java_delegate.obj()) {
- create_popup = Java_AwWebContentsDelegate_addNewContents(env,
- java_delegate.obj(), is_dialog, user_gesture);
+ create_popup = Java_AwWebContentsDelegate_addNewContents(
+ env, java_delegate, is_dialog, user_gesture);
}
if (create_popup) {
@@ -178,7 +178,7 @@ void AwWebContentsDelegate::NavigationStateChanged(
ScopedJavaLocalRef<jobject> java_delegate = GetJavaDelegate(env);
if (java_delegate.obj()) {
- Java_AwWebContentsDelegate_navigationStateChanged(env, java_delegate.obj(),
+ Java_AwWebContentsDelegate_navigationStateChanged(env, java_delegate,
changed_flags);
}
}
@@ -199,7 +199,7 @@ void AwWebContentsDelegate::CloseContents(WebContents* source) {
ScopedJavaLocalRef<jobject> java_delegate = GetJavaDelegate(env);
if (java_delegate.obj()) {
- Java_AwWebContentsDelegate_closeContents(env, java_delegate.obj());
+ Java_AwWebContentsDelegate_closeContents(env, java_delegate);
}
}
@@ -208,7 +208,7 @@ void AwWebContentsDelegate::ActivateContents(WebContents* contents) {
ScopedJavaLocalRef<jobject> java_delegate = GetJavaDelegate(env);
if (java_delegate.obj()) {
- Java_AwWebContentsDelegate_activateContents(env, java_delegate.obj());
+ Java_AwWebContentsDelegate_activateContents(env, java_delegate);
}
}
@@ -220,7 +220,7 @@ void AwWebContentsDelegate::LoadingStateChanged(WebContents* source,
ScopedJavaLocalRef<jobject> java_delegate = GetJavaDelegate(env);
if (java_delegate.obj()) {
- Java_AwWebContentsDelegate_loadingStateChanged(env, java_delegate.obj());
+ Java_AwWebContentsDelegate_loadingStateChanged(env, java_delegate);
}
}
« no previous file with comments | « android_webview/native/aw_quota_manager_bridge_impl.cc ('k') | android_webview/native/aw_web_resource_response_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698