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

Unified Diff: chrome/browser/password_manager/account_chooser_dialog_android.h

Issue 1509373004: Pass method parameters as JavaParamRef in chrome/browser (part 2). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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/password_manager/account_chooser_dialog_android.h
diff --git a/chrome/browser/password_manager/account_chooser_dialog_android.h b/chrome/browser/password_manager/account_chooser_dialog_android.h
index 5ab573f2707fa67e1af5eb2d7591243916aee74c..ed193caf6609e504733a10280f0894377ce7ac7e 100644
--- a/chrome/browser/password_manager/account_chooser_dialog_android.h
+++ b/chrome/browser/password_manager/account_chooser_dialog_android.h
@@ -31,21 +31,23 @@ class AccountChooserDialogAndroid {
const ManagePasswordsState::CredentialsCallback& callback);
~AccountChooserDialogAndroid();
- void Destroy(JNIEnv* env, jobject obj);
+ void Destroy(JNIEnv* env, const base::android::JavaParamRef<jobject>& obj);
void ShowDialog();
// Closes the dialog and propagates that no credentials was chosen.
- void CancelDialog(JNIEnv* env, jobject obj);
+ void CancelDialog(JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj);
// Propagates the credentials chosen by the user.
void OnCredentialClicked(JNIEnv* env,
- jobject obj,
+ const base::android::JavaParamRef<jobject>& obj,
jint credential_item,
jint credential_type);
// Opens new tab with page which explains the Smart Lock branding.
- void OnLinkClicked(JNIEnv* env, jobject obj);
+ void OnLinkClicked(JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj);
private:
const std::vector<const autofill::PasswordForm*>& local_credentials_forms()

Powered by Google App Engine
This is Rietveld 408576698