Update Confirmation UI for saved password change

chrome/browser/ui/passwords/manage_passwords_state.cc

 // Copyright 2015 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_state.h"
 
 #include "components/password_manager/core/browser/browser_save_password_progress_logger.h"
 #include "components/password_manager/core/browser/password_form_manager.h"
 #include "components/password_manager/core/browser/password_manager_client.h"
 #include "components/password_manager/core/common/credential_manager_types.h"
@@ skipping 102 matching lines @@
 }
 
 ManagePasswordsState::~ManagePasswordsState() {}
 
 void ManagePasswordsState::OnPendingPassword(
       scoped_ptr<password_manager::PasswordFormManager> form_manager) {
   ClearData();
   form_manager_ = form_manager.Pass();
   current_forms_weak_ = ScopedPtrMapToVector(form_manager_->best_matches());
   origin_ = form_manager_->pending_credentials().origin;
-  SetState(password_manager::ui::PENDING_PASSWORD_STATE);
+  if (client_->IsUpdatePasswordUIEnabled() &&
+      form_manager_->observed_form()
+          .IsPossibleChangePasswordFormWithoutUsername()) {

[vasilii, 2015/07/23 15:15:13]

This doesn't belong here. PasswordFormManager should think about update <->
save. In chrome/browser/ui/passwords these states should be isolated from each
other.

[dvadym, 2015/07/24 16:42:39]

Done.

+    SetState(password_manager::ui::PENDING_PASSWORD_UPDATE_STATE);
+  } else {
+    SetState(password_manager::ui::PENDING_PASSWORD_STATE);
+  }
 }
 
 void ManagePasswordsState::OnRequestCredentials(
     ScopedVector<autofill::PasswordForm> local_credentials,
     ScopedVector<autofill::PasswordForm> federated_credentials,
     const GURL& origin) {
   ClearData();
   local_credentials_forms_ = ConstifyVector(&local_credentials);
   federated_credentials_forms_ = ConstifyVector(&federated_credentials);
   origin_ = origin;
@@ skipping 142 matching lines @@
 void ManagePasswordsState::SetState(password_manager::ui::State state) {
   DCHECK(client_);
   if (client_->IsLoggingActive()) {
     password_manager::BrowserSavePasswordProgressLogger logger(client_);
     logger.LogNumber(
         autofill::SavePasswordProgressLogger::STRING_NEW_UI_STATE,
         state);
   }
   state_ = state;
 }