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

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

Issue 152683002: Passwords: Remove references to BrowserThread from PasswordStore. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix more tests. Created 6 years, 10 months 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_store_default.h" 5 #include "chrome/browser/password_manager/password_store_default.h"
6 6
7 #include <set> 7 #include <set>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/prefs/pref_service.h" 10 #include "base/prefs/pref_service.h"
11 #include "base/stl_util.h" 11 #include "base/stl_util.h"
12 #include "chrome/browser/chrome_notification_types.h" 12 #include "chrome/browser/chrome_notification_types.h"
13 #include "chrome/browser/password_manager/password_store_change.h" 13 #include "chrome/browser/password_manager/password_store_change.h"
14 #include "chrome/browser/profiles/profile.h" 14 #include "chrome/browser/profiles/profile.h"
15 #include "chrome/common/chrome_constants.h" 15 #include "chrome/common/chrome_constants.h"
16 #include "chrome/common/pref_names.h" 16 #include "chrome/common/pref_names.h"
17 #include "content/public/browser/browser_thread.h" 17 #include "content/public/browser/browser_thread.h"
18 #include "content/public/browser/notification_service.h" 18 #include "content/public/browser/notification_service.h"
19 19
20 using autofill::PasswordForm; 20 using autofill::PasswordForm;
21 using content::BrowserThread; 21 using content::BrowserThread;
22 22
23 PasswordStoreDefault::PasswordStoreDefault(LoginDatabase* login_db, 23 PasswordStoreDefault::PasswordStoreDefault(
24 Profile* profile) 24 scoped_refptr<base::SingleThreadTaskRunner> main_task_runner,
25 : login_db_(login_db), profile_(profile) { 25 scoped_refptr<base::SingleThreadTaskRunner> db_thread_runner,
26 LoginDatabase* login_db,
27 Profile* profile)
28 : PasswordStore(main_task_runner, db_thread_runner),
29 login_db_(login_db),
30 profile_(profile) {
26 DCHECK(login_db); 31 DCHECK(login_db);
27 DCHECK(profile); 32 DCHECK(profile);
28 } 33 }
29 34
30 PasswordStoreDefault::~PasswordStoreDefault() { 35 PasswordStoreDefault::~PasswordStoreDefault() {
31 } 36 }
32 37
33 void PasswordStoreDefault::ShutdownOnUIThread() { 38 void PasswordStoreDefault::ShutdownOnUIThread() {
34 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 39 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
35 profile_ = NULL; 40 profile_ = NULL;
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 std::vector<PasswordForm*>* forms) { 124 std::vector<PasswordForm*>* forms) {
120 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB)); 125 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
121 return login_db_->GetAutofillableLogins(forms); 126 return login_db_->GetAutofillableLogins(forms);
122 } 127 }
123 128
124 bool PasswordStoreDefault::FillBlacklistLogins( 129 bool PasswordStoreDefault::FillBlacklistLogins(
125 std::vector<PasswordForm*>* forms) { 130 std::vector<PasswordForm*>* forms) {
126 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB)); 131 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
127 return login_db_->GetBlacklistLogins(forms); 132 return login_db_->GetBlacklistLogins(forms);
128 } 133 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698