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

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, 11 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..6e228126e9788ac0c95e445cb9e34447d7acb491
--- /dev/null
+++ b/chrome/browser/password_manager/update_password_infobar_delegate.h
@@ -0,0 +1,82 @@
+// 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.
+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 {
+ return passwords_data_.GetCurrentForms();
+ }
+
+ // Returns the username of the saved credentials in case when there is only
+ // one credential pair stored.
+ base::string16 GetUsernameForOneAccountCase();
+
+ // Updates either pending credentials of form with |form_index| in case
+ // multiple
gone 2016/01/26 19:59:22 nit: Indentation is wonky.
melandory 2016/01/29 15:56:14 Done.
+ // choice available.
+ void UpdatePassword(int form_index);
+
+ // ConfirmInfoBarDelegate:
+ infobars::InfoBarDelegate::InfoBarIdentifier GetIdentifier() const override;
+ base::string16 GetButtonLabel(InfoBarButton button) const override;
+ bool Accept() override;
+ bool Cancel() override;
+
+ private:
+ UpdatePasswordInfoBarDelegate(
+ content::WebContents* web_contents,
+ scoped_ptr<password_manager::PasswordFormManager> form_to_update,
+ bool is_smartlock_branding_enabled);
+
+ // Updates the pending credentials.
+ void UpdatePasswordWithPending();
+
+ // Updates the form with index |form_index| in case of multiple credentials.
+ void UpdatePasswordWithCurrentForm(int form_index);
+
+ 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