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

Unified Diff: chrome/browser/ui/android/infobars/update_password_infobar.h

Issue 1490193003: [Password Manager] Update Confirmation UI for saved password change for Chrome on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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/ui/android/infobars/update_password_infobar.h
diff --git a/chrome/browser/ui/android/infobars/update_password_infobar.h b/chrome/browser/ui/android/infobars/update_password_infobar.h
new file mode 100644
index 0000000000000000000000000000000000000000..d7dac4453b9eeb9700600c7af6326b8ef1480bbd
--- /dev/null
+++ b/chrome/browser/ui/android/infobars/update_password_infobar.h
@@ -0,0 +1,37 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_ANDROID_INFOBARS_UPDATE_PASSWORD_INFOBAR_H_
+#define CHROME_BROWSER_UI_ANDROID_INFOBARS_UPDATE_PASSWORD_INFOBAR_H_
+
+#include "base/android/scoped_java_ref.h"
+#include "base/macros.h"
+#include "chrome/browser/password_manager/update_password_infobar_delegate.h"
Peter Kasting 2016/02/13 01:44:52 Nit: I think you can forward-declare UpdatePasswor
melandory 2016/02/22 12:41:09 Done.
+#include "chrome/browser/ui/android/infobars/confirm_infobar.h"
+
+// The Android infobar that offers the user the ability to update a password
+// for the site.
Peter Kasting 2016/02/13 01:44:52 Nit: How about: The infobar to be used with Updat
melandory 2016/02/22 12:41:09 Done.
+class UpdatePasswordInfoBar : public ConfirmInfoBar {
+ public:
+ static bool Register(JNIEnv* env);
Peter Kasting 2016/02/13 01:44:52 Nit: Google style guide says static methods go bel
melandory 2016/02/22 12:41:09 Done.
+
+ explicit UpdatePasswordInfoBar(
+ scoped_ptr<UpdatePasswordInfoBarDelegate> delegate);
+ ~UpdatePasswordInfoBar() override;
+
+ int GetIdOfSelectedUsername();
Peter Kasting 2016/02/13 01:44:52 Nit: const?
melandory 2016/02/22 12:41:09 Done.
+
+ private:
+ // ConfirmInfoBar:
+ base::android::ScopedJavaLocalRef<jobject> CreateRenderInfoBar(
+ JNIEnv* env) override;
+ void OnLinkClicked(JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj) override;
+
+ base::android::ScopedJavaGlobalRef<jobject> java_infobar_;
+
+ DISALLOW_COPY_AND_ASSIGN(UpdatePasswordInfoBar);
+};
+
+#endif // CHROME_BROWSER_UI_ANDROID_INFOBARS_UPDATE_PASSWORD_INFOBAR_H_

Powered by Google App Engine
This is Rietveld 408576698