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

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: Separate dummy log manager 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/log_manager.h"
8 #include "components/password_manager/core/browser/password_form_manager.h" 9 #include "components/password_manager/core/browser/password_form_manager.h"
10 #include "components/password_manager/core/browser/password_manager.h"
9 #include "components/password_manager/core/browser/password_manager_client.h" 11 #include "components/password_manager/core/browser/password_manager_client.h"
10 #include "components/password_manager/core/common/credential_manager_types.h" 12 #include "components/password_manager/core/common/credential_manager_types.h"
11 13
12 using password_manager::PasswordFormManager; 14 using password_manager::PasswordFormManager;
13 using autofill::PasswordFormMap; 15 using autofill::PasswordFormMap;
14 16
15 namespace { 17 namespace {
16 18
17 // Returns a vector containing the values of a map. 19 // Returns a vector containing the values of a map.
18 template <typename Map> 20 template <typename Map>
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
286 } 288 }
287 289
288 void ManagePasswordsState::DeleteForm(const autofill::PasswordForm& form) { 290 void ManagePasswordsState::DeleteForm(const autofill::PasswordForm& form) {
289 RemoveFormFromVector(form, &current_forms_weak_); 291 RemoveFormFromVector(form, &current_forms_weak_);
290 RemoveFormFromVector(form, &local_credentials_forms_); 292 RemoveFormFromVector(form, &local_credentials_forms_);
291 RemoveFormFromVector(form, &federated_credentials_forms_); 293 RemoveFormFromVector(form, &federated_credentials_forms_);
292 } 294 }
293 295
294 void ManagePasswordsState::SetState(password_manager::ui::State state) { 296 void ManagePasswordsState::SetState(password_manager::ui::State state) {
295 DCHECK(client_); 297 DCHECK(client_);
296 if (client_->IsLoggingActive()) { 298 if (client_->GetLogManager()->IsLoggingActive()) {
297 password_manager::BrowserSavePasswordProgressLogger logger(client_); 299 password_manager::BrowserSavePasswordProgressLogger logger(
300 client_->GetLogManager());
298 logger.LogNumber( 301 logger.LogNumber(
299 autofill::SavePasswordProgressLogger::STRING_NEW_UI_STATE, 302 autofill::SavePasswordProgressLogger::STRING_NEW_UI_STATE,
300 state); 303 state);
301 } 304 }
302 state_ = state; 305 state_ = state;
303 } 306 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698