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

Unified Diff: chrome/browser/ui/passwords/manage_passwords_ui_controller.cc

Issue 2915763003: [Password Manager] Show omnibox icon and anchored prompt once user start typing password (Closed)
Patch Set: Sent For Review Created 3 years, 5 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/passwords/manage_passwords_ui_controller.cc
diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc b/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
index 6ef8efa9463e80cf8cc80a17696e21889dadbcb4..97a832d0a66cee8f40741cfa929b1c795e0e97ba 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller.cc
@@ -94,6 +94,31 @@ void ManagePasswordsUIController::OnUpdatePasswordSubmitted(
UpdateBubbleAndIconVisibility();
}
+void ManagePasswordsUIController::OnShowManualFallback(
+ std::unique_ptr<PasswordFormManager> form_manager,
+ bool is_update) {
+ DestroyAccountChooser();
+ if (form_manager->has_generated_password())
+ passwords_data_.OnAutomaticPasswordSave(std::move(form_manager));
vasilii 2017/07/21 12:48:19 That I don't understand. What scenario do you keep
kolos1 2017/07/24 15:33:29 If a user accepts a generated password, the icon a
vasilii 2017/07/25 11:17:46 That password is saved automatically and the confi
kolos1 2017/07/26 13:40:57 As discussed in person, let's put the question to
+ else if (is_update)
+ passwords_data_.OnUpdatePassword(std::move(form_manager));
+ else
+ passwords_data_.OnPendingPassword(std::move(form_manager));
+ UpdateBubbleAndIconVisibility();
+}
+
+void ManagePasswordsUIController::OnHideManualFallback() {
+ if (passwords_data_.state() != password_manager::ui::PENDING_PASSWORD_STATE &&
+ passwords_data_.state() !=
+ password_manager::ui::PENDING_PASSWORD_UPDATE_STATE &&
+ passwords_data_.state() != password_manager::ui::CONFIRMATION_STATE) {
+ return;
+ }
+
+ passwords_data_.TransitionToState(password_manager::ui::MANAGE_STATE);
+ UpdateBubbleAndIconVisibility();
+}
+
bool ManagePasswordsUIController::OnChooseCredentials(
std::vector<std::unique_ptr<autofill::PasswordForm>> local_credentials,
const GURL& origin,

Powered by Google App Engine
This is Rietveld 408576698