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

Unified Diff: chrome/browser/android/ntp/offline_page_suggestions_provider_factory.cc

Issue 2205233002: Combine all suggestions factories into ContentSuggestionsServiceFactory (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Bernhard's comments Created 4 years, 4 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
Index: chrome/browser/android/ntp/offline_page_suggestions_provider_factory.cc
diff --git a/chrome/browser/android/ntp/offline_page_suggestions_provider_factory.cc b/chrome/browser/android/ntp/offline_page_suggestions_provider_factory.cc
deleted file mode 100644
index ea1dfae9d1dcdbeba27c5aae2ef78739e9b94060..0000000000000000000000000000000000000000
--- a/chrome/browser/android/ntp/offline_page_suggestions_provider_factory.cc
+++ /dev/null
@@ -1,76 +0,0 @@
-// Copyright 2016 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/android/ntp/offline_page_suggestions_provider_factory.h"
-
-#include "base/feature_list.h"
-#include "base/memory/singleton.h"
-#include "chrome/browser/android/chrome_feature_list.h"
-#include "chrome/browser/android/offline_pages/offline_page_model_factory.h"
-#include "chrome/browser/ntp_snippets/content_suggestions_service_factory.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/common/pref_names.h"
-#include "components/keyed_service/content/browser_context_dependency_manager.h"
-#include "components/ntp_snippets/content_suggestions_service.h"
-#include "components/ntp_snippets/offline_pages/offline_page_suggestions_provider.h"
-#include "components/offline_pages/offline_page_model.h"
-#include "components/prefs/pref_service.h"
-#include "content/public/browser/browser_context.h"
-
-using ntp_snippets::ContentSuggestionsService;
-using ntp_snippets::OfflinePageSuggestionsProvider;
-using offline_pages::OfflinePageModel;
-using offline_pages::OfflinePageModelFactory;
-
-// static
-OfflinePageSuggestionsProviderFactory*
-OfflinePageSuggestionsProviderFactory::GetInstance() {
- return base::Singleton<OfflinePageSuggestionsProviderFactory>::get();
-}
-
-// static
-ntp_snippets::OfflinePageSuggestionsProvider*
-OfflinePageSuggestionsProviderFactory::GetForProfile(Profile* profile) {
- DCHECK(!profile->IsOffTheRecord());
- return static_cast<ntp_snippets::OfflinePageSuggestionsProvider*>(
- GetInstance()->GetServiceForBrowserContext(profile, true));
-}
-
-OfflinePageSuggestionsProviderFactory::OfflinePageSuggestionsProviderFactory()
- : BrowserContextKeyedServiceFactory(
- "OfflinePageSuggestionsProvider",
- BrowserContextDependencyManager::GetInstance()) {
- DependsOn(ContentSuggestionsServiceFactory::GetInstance());
- DependsOn(OfflinePageModelFactory::GetInstance());
-}
-
-OfflinePageSuggestionsProviderFactory::
- ~OfflinePageSuggestionsProviderFactory() {}
-
-KeyedService* OfflinePageSuggestionsProviderFactory::BuildServiceInstanceFor(
- content::BrowserContext* context) const {
- Profile* profile = Profile::FromBrowserContext(context);
-
- ContentSuggestionsService* content_suggestions_service =
- ContentSuggestionsServiceFactory::GetForProfile(profile);
- if (content_suggestions_service->state() ==
- ContentSuggestionsService::State::DISABLED) {
- return nullptr;
- }
-
- if (!base::FeatureList::IsEnabled(
- chrome::android::kNTPOfflinePageSuggestionsFeature)) {
- return nullptr;
- }
-
- OfflinePageModel* offline_page_model =
- OfflinePageModelFactory::GetForBrowserContext(profile);
-
- OfflinePageSuggestionsProvider* offline_page_suggestions_provider =
- new OfflinePageSuggestionsProvider(
- content_suggestions_service->category_factory(), offline_page_model);
- content_suggestions_service->RegisterProvider(
- offline_page_suggestions_provider);
- return offline_page_suggestions_provider;
-}

Powered by Google App Engine
This is Rietveld 408576698