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

Unified Diff: chrome/browser/password_manager/update_password_infobar_delegate.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/password_manager/update_password_infobar_delegate.h
diff --git a/chrome/browser/password_manager/update_password_infobar_delegate.h b/chrome/browser/password_manager/update_password_infobar_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..19bb17b204d21cdf085f59244d5e06e95009e2dd
--- /dev/null
+++ b/chrome/browser/password_manager/update_password_infobar_delegate.h
@@ -0,0 +1,68 @@
+// 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_PASSWORD_MANAGER_UPDATE_PASSWORD_INFOBAR_DELEGATE_H_
+#define CHROME_BROWSER_PASSWORD_MANAGER_UPDATE_PASSWORD_INFOBAR_DELEGATE_H_
+
+#include <vector>
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/password_manager/password_manager_infobar_delegate.h"
+#include "chrome/browser/ui/passwords/manage_passwords_state.h"
+#include "components/password_manager/core/browser/password_form_manager.h"
+
+namespace content {
+class WebContents;
+}
+
+// If PasswordManager encounters the login which is already known or user fills
+// the password change form, then user is prompted with an infobar asking if
+// already saved credentials should be updated or not.
+// In case several credentials are stored for the web site, user can choose
+// which one should be updated.
Peter Kasting 2016/02/13 01:44:52 Nit: Grammar. How about: An infobar delegate whi
melandory 2016/02/22 12:41:09 Done.
+class UpdatePasswordInfoBarDelegate : public PasswordManagerInfoBarDelegate {
+ public:
+ static void Create(
+ content::WebContents* web_contents,
+ scoped_ptr<password_manager::PasswordFormManager> form_to_update);
+
+ ~UpdatePasswordInfoBarDelegate() override;
+
+ base::string16 branding() const { return branding_; }
+ bool is_smartlock_branding_enabled() const {
+ return is_smartlock_branding_enabled_;
+ }
+
+ // Returns true if infobar for the multi credentials case should be shown, e.g
+ // user changing a password on a password change form and has several
+ // credentials save for the web site.
+ bool ShowMultipleAccounts() const;
+
+ const std::vector<const autofill::PasswordForm*>& GetCurrentForms() const;
+
+ // Returns the username of the saved credentials in the case when there is
+ // only one credential pair stored.
+ base::string16 GetUsernameForOneAccountCase();
Peter Kasting 2016/02/13 01:44:52 Nit: One -> Single
melandory 2016/02/22 12:41:09 Done.
+
+ private:
+ UpdatePasswordInfoBarDelegate(
+ content::WebContents* web_contents,
+ scoped_ptr<password_manager::PasswordFormManager> form_to_update,
+ bool is_smartlock_branding_enabled);
+
+ // ConfirmInfoBarDelegate:
+ infobars::InfoBarDelegate::InfoBarIdentifier GetIdentifier() const override;
+ base::string16 GetButtonLabel(InfoBarButton button) const override;
+ bool Accept() override;
+ bool Cancel() override;
+
+ ManagePasswordsState passwords_data_;
+ base::string16 branding_;
+ bool is_smartlock_branding_enabled_;
+
+ DISALLOW_COPY_AND_ASSIGN(UpdatePasswordInfoBarDelegate);
+};
+
+#endif // CHROME_BROWSER_PASSWORD_MANAGER_UPDATE_PASSWORD_INFOBAR_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698