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

Unified Diff: components/browsing_data/core/counters/passwords_counter.cc

Issue 2153863002: Move counters for passwords, history and autofill to components (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@separate_build_targets_in_components_bd
Patch Set: Addressed comments Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/browsing_data/core/counters/passwords_counter.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/browsing_data/core/counters/passwords_counter.cc
diff --git a/chrome/browser/browsing_data/passwords_counter.cc b/components/browsing_data/core/counters/passwords_counter.cc
similarity index 61%
rename from chrome/browser/browsing_data/passwords_counter.cc
rename to components/browsing_data/core/counters/passwords_counter.cc
index 5af21741049cb9735eef083b7b87a1a370ea6ee2..ab60368ca17ee346deac1e927d8e4a94c8bbc625 100644
--- a/chrome/browser/browsing_data/passwords_counter.cc
+++ b/components/browsing_data/core/counters/passwords_counter.cc
@@ -1,28 +1,30 @@
-// Copyright (c) 2015 The Chromium Authors. All rights reserved.
+// 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/browsing_data/passwords_counter.h"
-#include "chrome/browser/password_manager/password_store_factory.h"
-#include "chrome/browser/profiles/profile.h"
+#include "components/browsing_data/core/counters/passwords_counter.h"
+
#include "components/browsing_data/core/pref_names.h"
#include "components/password_manager/core/browser/password_store.h"
-PasswordsCounter::PasswordsCounter(Profile* profile)
- : BrowsingDataCounter(browsing_data::prefs::kDeletePasswords),
- profile_(profile) {}
+namespace browsing_data {
+
+PasswordsCounter::PasswordsCounter(
+ scoped_refptr<password_manager::PasswordStore> store) : store_(store) {}
PasswordsCounter::~PasswordsCounter() {
store_->RemoveObserver(this);
}
void PasswordsCounter::OnInitialized() {
- store_ = PasswordStoreFactory::GetForProfile(
- profile_, ServiceAccessType::EXPLICIT_ACCESS);
DCHECK(store_);
store_->AddObserver(this);
}
+const char* PasswordsCounter::GetPrefName() const {
+ return browsing_data::prefs::kDeletePasswords;
+}
+
void PasswordsCounter::Count() {
cancelable_task_tracker()->TryCancelAll();
// TODO(msramek): We don't actually need the logins themselves, just their
@@ -35,15 +37,15 @@ void PasswordsCounter::Count() {
void PasswordsCounter::OnGetPasswordStoreResults(
ScopedVector<autofill::PasswordForm> results) {
base::Time start = GetPeriodStart();
- ReportResult(std::count_if(
- results.begin(),
- results.end(),
- [start](const autofill::PasswordForm* form) {
- return form->date_created >= start;
- }));
+ ReportResult(std::count_if(results.begin(), results.end(),
+ [start](const autofill::PasswordForm* form) {
+ return form->date_created >= start;
+ }));
}
void PasswordsCounter::OnLoginsChanged(
const password_manager::PasswordStoreChangeList& changes) {
Restart();
}
+
+} // namespace browsing_data
« no previous file with comments | « components/browsing_data/core/counters/passwords_counter.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698