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

Unified Diff: components/ntp_tiles/most_visited_sites_unittest.cc

Issue 2946143002: Remove legacy popular sites cache cleaning (Closed)
Patch Set: Remove unused imports Created 3 years, 6 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 | « chrome/browser/ntp_tiles/chrome_popular_sites_factory.cc ('k') | components/ntp_tiles/popular_sites_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/ntp_tiles/most_visited_sites_unittest.cc
diff --git a/components/ntp_tiles/most_visited_sites_unittest.cc b/components/ntp_tiles/most_visited_sites_unittest.cc
index 09c16d9249c2afc53e8ad98e2ed7e7726a83b888..6f2f895f09fdeae561ef8c35e563a0636f499679 100644
--- a/components/ntp_tiles/most_visited_sites_unittest.cc
+++ b/components/ntp_tiles/most_visited_sites_unittest.cc
@@ -21,7 +21,6 @@
#include "base/strings/utf_string_conversions.h"
#include "base/task/cancelable_task_tracker.h"
#include "base/test/scoped_feature_list.h"
-#include "base/test/sequenced_worker_pool_owner.h"
#include "base/threading/thread_task_runner_handle.h"
#include "components/history/core/browser/top_sites.h"
#include "components/history/core/browser/top_sites_observer.h"
@@ -259,8 +258,7 @@ class PopularSitesFactoryForTest {
: prefs_(pref_service),
url_fetcher_factory_(/*default_factory=*/nullptr),
url_request_context_(new net::TestURLRequestContextGetter(
- base::ThreadTaskRunnerHandle::Get())),
- worker_pool_owner_(/*max_threads=*/2, "PopularSitesFactoryForTest.") {
+ base::ThreadTaskRunnerHandle::Get())) {
PopularSitesImpl::RegisterProfilePrefs(pref_service->registry());
if (enabled) {
prefs_->SetString(prefs::kPopularSitesOverrideCountry, "IN");
@@ -285,17 +283,16 @@ class PopularSitesFactoryForTest {
std::unique_ptr<PopularSites> New() {
return base::MakeUnique<PopularSitesImpl>(
- worker_pool_owner_.pool().get(), prefs_,
+ prefs_,
/*template_url_service=*/nullptr,
/*variations_service=*/nullptr, url_request_context_.get(),
- /*directory=*/base::FilePath(), base::Bind(JsonUnsafeParser::Parse));
+ base::Bind(JsonUnsafeParser::Parse));
}
private:
PrefService* prefs_;
net::FakeURLFetcherFactory url_fetcher_factory_;
scoped_refptr<net::TestURLRequestContextGetter> url_request_context_;
- base::SequencedWorkerPoolOwner worker_pool_owner_;
};
// CallbackList-like container without Subscription, mimicking the
« no previous file with comments | « chrome/browser/ntp_tiles/chrome_popular_sites_factory.cc ('k') | components/ntp_tiles/popular_sites_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698