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

Side by Side Diff: chrome/browser/password_manager/password_manager.cc

Issue 11316163: Remove the last usages of PrefObserver outside of Prefs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review comments. Created 8 years 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/password_manager/password_manager.h" 5 #include "chrome/browser/password_manager/password_manager.h"
6 6
7 #include "base/metrics/histogram.h" 7 #include "base/metrics/histogram.h"
8 #include "base/threading/platform_thread.h" 8 #include "base/threading/platform_thread.h"
9 #include "base/string_util.h" 9 #include "base/string_util.h"
10 #include "base/utf_string_conversions.h" 10 #include "base/utf_string_conversions.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 new PasswordManager(contents, delegate)); 79 new PasswordManager(contents, delegate));
80 } 80 }
81 81
82 PasswordManager::PasswordManager(WebContents* web_contents, 82 PasswordManager::PasswordManager(WebContents* web_contents,
83 PasswordManagerDelegate* delegate) 83 PasswordManagerDelegate* delegate)
84 : content::WebContentsObserver(web_contents), 84 : content::WebContentsObserver(web_contents),
85 delegate_(delegate), 85 delegate_(delegate),
86 observer_(NULL) { 86 observer_(NULL) {
87 DCHECK(delegate_); 87 DCHECK(delegate_);
88 password_manager_enabled_.Init(prefs::kPasswordManagerEnabled, 88 password_manager_enabled_.Init(prefs::kPasswordManagerEnabled,
89 delegate_->GetProfile()->GetPrefs(), NULL); 89 delegate_->GetProfile()->GetPrefs());
90 90
91 ReportMetrics(*password_manager_enabled_); 91 ReportMetrics(*password_manager_enabled_);
92 } 92 }
93 93
94 PasswordManager::~PasswordManager() { 94 PasswordManager::~PasswordManager() {
95 } 95 }
96 96
97 void PasswordManager::SetFormHasGeneratedPassword(const PasswordForm& form) { 97 void PasswordManager::SetFormHasGeneratedPassword(const PasswordForm& form) {
98 for (ScopedVector<PasswordFormManager>::iterator iter = 98 for (ScopedVector<PasswordFormManager>::iterator iter =
99 pending_login_managers_.begin(); 99 pending_login_managers_.begin();
(...skipping 207 matching lines...) Expand 10 before | Expand all | Expand 10 after
307 delegate_->FillPasswordForm(fill_data); 307 delegate_->FillPasswordForm(fill_data);
308 return; 308 return;
309 } 309 }
310 default: 310 default:
311 if (observer_) { 311 if (observer_) {
312 observer_->OnAutofillDataAvailable(preferred_match.username_value, 312 observer_->OnAutofillDataAvailable(preferred_match.username_value,
313 preferred_match.password_value); 313 preferred_match.password_value);
314 } 314 }
315 } 315 }
316 } 316 }
OLDNEW
« no previous file with comments | « chrome/browser/net/net_error_tab_helper.cc ('k') | chrome/browser/plugins/plugin_status_pref_setter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698