Update Confirmation UI for saved password change

chrome/browser/ui/passwords/manage_passwords_bubble_model.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/ui/passwords/manage_passwords_bubble_model.h"
 
 #include "base/command_line.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/password_manager/password_store_factory.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 229 matching lines @@
 
 void ManagePasswordsBubbleModel::OnSaveClicked() {
   dismissal_reason_ = metrics_util::CLICKED_SAVE;
   RecordExperimentStatistics(web_contents(), dismissal_reason_);
   ManagePasswordsUIController* manage_passwords_ui_controller =
       ManagePasswordsUIController::FromWebContents(web_contents());
   manage_passwords_ui_controller->SavePassword();
   state_ = password_manager::ui::MANAGE_STATE;
 }
 
+void ManagePasswordsBubbleModel::OnUpdateClicked(
+    const autofill::PasswordForm& password_form) {
+  dismissal_reason_ = metrics_util::CLICKED_UPDATE;
+  RecordExperimentStatistics(web_contents(), dismissal_reason_);
+  ManagePasswordsUIController* manage_passwords_ui_controller =
+      ManagePasswordsUIController::FromWebContents(web_contents());
+  manage_passwords_ui_controller->UpdatePassword(password_form);
+  state_ = password_manager::ui::MANAGE_STATE;
+}
+
 void ManagePasswordsBubbleModel::OnDoneClicked() {
   dismissal_reason_ = metrics_util::CLICKED_DONE;
 }
 
 // TODO(gcasto): Is it worth having this be separate from OnDoneClicked()?
 // User intent is pretty similar in both cases.
 void ManagePasswordsBubbleModel::OnOKClicked() {
   dismissal_reason_ = metrics_util::CLICKED_OK;
 }
 
@@ skipping 68 matching lines @@
   return GetFieldWidth(USERNAME_FIELD);
 }
 
 // static
 int ManagePasswordsBubbleModel::PasswordFieldWidth() {
   return GetFieldWidth(PASSWORD_FIELD);
 }
 
 void ManagePasswordsBubbleModel::UpdatePendingStateTitle() {
   title_brand_link_range_ = gfx::Range();
+  int title_message_id =
+      pending_password_.is_password_change_form_without_username
+          ? IDS_UPDATE_PASSWORD
+          : IDS_SAVE_PASSWORD;
   if (never_save_passwords_) {
     title_ = l10n_util::GetStringUTF16(
         IDS_MANAGE_PASSWORDS_BLACKLIST_CONFIRMATION_TITLE);
   } else if (IsSmartLockBrandingEnabled(GetProfile())) {
     // "Google Smart Lock" should be a hyperlink.
     base::string16 brand_link =
         l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_SMART_LOCK);
     size_t offset = 0;
-    title_ = l10n_util::GetStringFUTF16(IDS_SAVE_PASSWORD, brand_link, &offset);
+    title_ = l10n_util::GetStringFUTF16(title_message_id, brand_link, &offset);
     title_brand_link_range_ = gfx::Range(offset, offset + brand_link.length());
   } else {
     base::string16 brand_link =
         l10n_util::GetStringUTF16(IDS_SAVE_PASSWORD_TITLE_BRAND);
-    title_ = l10n_util::GetStringFUTF16(IDS_SAVE_PASSWORD, brand_link);
+    title_ = l10n_util::GetStringFUTF16(title_message_id, brand_link);
   }
+  save_button_text_ = l10n_util::GetStringUTF16(

[vasilii, 2015/06/03 18:32:01]

This should go to the constructor.

[dvadym, 2015/06/19 15:27:29]

Done.

+      pending_password_.is_password_change_form_without_username
+          ? IDS_PASSWORD_MANAGER_UPDATE_BUTTON
+          : IDS_PASSWORD_MANAGER_SAVE_BUTTON);
 }