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

Side by Side Diff: chrome/browser/ui/passwords/manage_passwords_state.cc

Issue 1415533013: Fix password manager internals renderer reporting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 5 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/passwords/manage_passwords_state.h" 5 #include "chrome/browser/ui/passwords/manage_passwords_state.h"
6 6
7 #include "components/password_manager/core/browser/browser_save_password_progres s_logger.h" 7 #include "components/password_manager/core/browser/browser_save_password_progres s_logger.h"
8 #include "components/password_manager/core/browser/password_form_manager.h" 8 #include "components/password_manager/core/browser/password_form_manager.h"
9 #include "components/password_manager/core/browser/password_manager.h"
9 #include "components/password_manager/core/browser/password_manager_client.h" 10 #include "components/password_manager/core/browser/password_manager_client.h"
10 #include "components/password_manager/core/common/credential_manager_types.h" 11 #include "components/password_manager/core/common/credential_manager_types.h"
11 12
12 using password_manager::PasswordFormManager; 13 using password_manager::PasswordFormManager;
13 using autofill::PasswordFormMap; 14 using autofill::PasswordFormMap;
14 15
15 namespace { 16 namespace {
16 17
17 // Returns a vector containing the values of a map. 18 // Returns a vector containing the values of a map.
18 template <typename Map> 19 template <typename Map>
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
286 } 287 }
287 288
288 void ManagePasswordsState::DeleteForm(const autofill::PasswordForm& form) { 289 void ManagePasswordsState::DeleteForm(const autofill::PasswordForm& form) {
289 RemoveFormFromVector(form, &current_forms_weak_); 290 RemoveFormFromVector(form, &current_forms_weak_);
290 RemoveFormFromVector(form, &local_credentials_forms_); 291 RemoveFormFromVector(form, &local_credentials_forms_);
291 RemoveFormFromVector(form, &federated_credentials_forms_); 292 RemoveFormFromVector(form, &federated_credentials_forms_);
292 } 293 }
293 294
294 void ManagePasswordsState::SetState(password_manager::ui::State state) { 295 void ManagePasswordsState::SetState(password_manager::ui::State state) {
295 DCHECK(client_); 296 DCHECK(client_);
296 if (client_->IsLoggingActive()) { 297 if (client_->GetPasswordManager()->IsLoggingActive()) {
297 password_manager::BrowserSavePasswordProgressLogger logger(client_); 298 password_manager::BrowserSavePasswordProgressLogger logger(
299 client_->GetPasswordManager());
298 logger.LogNumber( 300 logger.LogNumber(
299 autofill::SavePasswordProgressLogger::STRING_NEW_UI_STATE, 301 autofill::SavePasswordProgressLogger::STRING_NEW_UI_STATE,
300 state); 302 state);
301 } 303 }
302 state_ = state; 304 state_ = state;
303 } 305 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698