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

Unified Diff: components/web_contents_delegate_android/validation_message_bubble_android.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: components/web_contents_delegate_android/validation_message_bubble_android.cc
diff --git a/components/web_contents_delegate_android/validation_message_bubble_android.cc b/components/web_contents_delegate_android/validation_message_bubble_android.cc
index 9223540320cf18af3defae9d73b18ff29cd6d67f..0dd5eee1628c481d2806b6199376a52b24609c86 100644
--- a/components/web_contents_delegate_android/validation_message_bubble_android.cc
+++ b/components/web_contents_delegate_android/validation_message_bubble_android.cc
@@ -43,19 +43,16 @@ ValidationMessageBubbleAndroid::ValidationMessageBubbleAndroid(
JNIEnv* env = base::android::AttachCurrentThread();
java_validation_message_bubble_.Reset(
Java_ValidationMessageBubble_createAndShow(
- env,
- java_content_view_core.obj(),
- anchor_in_root_view.x(),
- anchor_in_root_view.y(),
- anchor_in_root_view.width(),
+ env, java_content_view_core, anchor_in_root_view.x(),
+ anchor_in_root_view.y(), anchor_in_root_view.width(),
anchor_in_root_view.height(),
- ConvertUTF16ToJavaString(env, main_text).obj(),
- ConvertUTF16ToJavaString(env, sub_text).obj()));
+ ConvertUTF16ToJavaString(env, main_text),
+ ConvertUTF16ToJavaString(env, sub_text)));
}
ValidationMessageBubbleAndroid::~ValidationMessageBubbleAndroid() {
Java_ValidationMessageBubble_close(base::android::AttachCurrentThread(),
- java_validation_message_bubble_.obj());
+ java_validation_message_bubble_);
}
void ValidationMessageBubbleAndroid::SetPositionRelativeToAnchor(
@@ -66,13 +63,9 @@ void ValidationMessageBubbleAndroid::SetPositionRelativeToAnchor(
return;
Java_ValidationMessageBubble_setPositionRelativeToAnchor(
- base::android::AttachCurrentThread(),
- java_validation_message_bubble_.obj(),
- java_content_view_core.obj(),
- anchor_in_root_view.x(),
- anchor_in_root_view.y(),
- anchor_in_root_view.width(),
- anchor_in_root_view.height());
+ base::android::AttachCurrentThread(), java_validation_message_bubble_,
+ java_content_view_core, anchor_in_root_view.x(), anchor_in_root_view.y(),
+ anchor_in_root_view.width(), anchor_in_root_view.height());
}
} // namespace web_contents_delegate_android

Powered by Google App Engine
This is Rietveld 408576698