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

Unified Diff: chrome/browser/browsing_data/autofill_counter.h

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 | « no previous file | chrome/browser/browsing_data/autofill_counter.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browsing_data/autofill_counter.h
diff --git a/chrome/browser/browsing_data/autofill_counter.h b/chrome/browser/browsing_data/autofill_counter.h
deleted file mode 100644
index bc65dc332763a2269d70c3b22145c7557f7e5d2a..0000000000000000000000000000000000000000
--- a/chrome/browser/browsing_data/autofill_counter.h
+++ /dev/null
@@ -1,88 +0,0 @@
-// 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.
-
-#ifndef CHROME_BROWSER_BROWSING_DATA_AUTOFILL_COUNTER_H_
-#define CHROME_BROWSER_BROWSING_DATA_AUTOFILL_COUNTER_H_
-
-#include "base/macros.h"
-#include "base/threading/thread_checker.h"
-#include "base/time/time.h"
-#include "components/browsing_data/core/counters/browsing_data_counter.h"
-#include "components/webdata/common/web_data_service_consumer.h"
-
-class Profile;
-
-namespace autofill {
-class AutofillWebDataService;
-}
-
-class AutofillCounter : public browsing_data::BrowsingDataCounter,
- public WebDataServiceConsumer {
- public:
- class AutofillResult : public FinishedResult {
- public:
- AutofillResult(const AutofillCounter* source,
- ResultInt num_suggestions,
- ResultInt num_credit_cards,
- ResultInt num_addresses);
- ~AutofillResult() override;
-
- ResultInt num_credit_cards() const { return num_credit_cards_; }
- ResultInt num_addresses() const { return num_addresses_; }
-
- private:
- ResultInt num_credit_cards_;
- ResultInt num_addresses_;
-
- DISALLOW_COPY_AND_ASSIGN(AutofillResult);
- };
-
- explicit AutofillCounter(Profile* profile);
- ~AutofillCounter() override;
-
- // BrowsingDataCounter implementation.
- void OnInitialized() override;
-
- // Whether the counting is in progress.
- bool HasPendingQuery() {
- return suggestions_query_ || credit_cards_query_ || addresses_query_;
- }
-
- // Set the beginning of the time period for testing. AutofillTable does not
- // allow us to set time explicitly, and BrowsingDataCounter recognizes
- // only predefined time periods, out of which the lowest one is one hour.
- // Obviously, the test cannot run that long.
- // TODO(msramek): Consider changing BrowsingDataCounter to use arbitrary
- // time periods instead of BrowsingDataRemover::TimePeriod.
- void SetPeriodStartForTesting(const base::Time& period_start_for_testing);
-
- private:
- Profile* profile_;
- base::ThreadChecker thread_checker_;
-
- scoped_refptr<autofill::AutofillWebDataService> web_data_service_;
-
- WebDataServiceBase::Handle suggestions_query_;
- WebDataServiceBase::Handle credit_cards_query_;
- WebDataServiceBase::Handle addresses_query_;
-
- ResultInt num_suggestions_;
- ResultInt num_credit_cards_;
- ResultInt num_addresses_;
-
- base::Time period_start_for_testing_;
-
- void Count() override;
-
- // WebDataServiceConsumer implementation.
- void OnWebDataServiceRequestDone(WebDataServiceBase::Handle handle,
- const WDTypedResult* result) override;
-
- // Cancel all pending requests to AutofillWebdataService.
- void CancelAllRequests();
-
- DISALLOW_COPY_AND_ASSIGN(AutofillCounter);
-};
-
-#endif // CHROME_BROWSER_BROWSING_DATA_AUTOFILL_COUNTER_H_
« no previous file with comments | « no previous file | chrome/browser/browsing_data/autofill_counter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698