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

Unified Diff: chrome/browser/ui/android/javascript_app_modal_dialog_android.h

Issue 1481553002: jni: Pass method parameters as JavaParamRef in chrome/browser/ui. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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/ui/android/javascript_app_modal_dialog_android.h
diff --git a/chrome/browser/ui/android/javascript_app_modal_dialog_android.h b/chrome/browser/ui/android/javascript_app_modal_dialog_android.h
index bef1465583de6e364eb857855cc900d44084bab5..b6a6fc096d31f23fae86bc6657cab99db1330b03 100644
--- a/chrome/browser/ui/android/javascript_app_modal_dialog_android.h
+++ b/chrome/browser/ui/android/javascript_app_modal_dialog_android.h
@@ -30,11 +30,14 @@ class JavascriptAppModalDialogAndroid
bool IsShowing() const override;
// Called when java confirms or cancels the dialog.
- void DidAcceptAppModalDialog(JNIEnv* env,
- jobject obj,
- jstring prompt_text,
+ void DidAcceptAppModalDialog(
+ JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj,
+ const base::android::JavaParamRef<jstring>& prompt_text,
+ bool suppress_js_dialogs);
+ void DidCancelAppModalDialog(JNIEnv* env,
+ const base::android::JavaParamRef<jobject>&,
bool suppress_js_dialogs);
- void DidCancelAppModalDialog(JNIEnv* env, jobject, bool suppress_js_dialogs);
const base::android::ScopedJavaGlobalRef<jobject>& GetDialogObject() const;

Powered by Google App Engine
This is Rietveld 408576698