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

Unified Diff: chrome/browser/dom_distiller/dom_distiller_service_factory.cc

Issue 254483003: Start requiring DistillerPage for calls to DomDistillerService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Indent fixes (full git cl format) Created 6 years, 8 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/dom_distiller/dom_distiller_service_factory.cc
diff --git a/chrome/browser/dom_distiller/dom_distiller_service_factory.cc b/chrome/browser/dom_distiller/dom_distiller_service_factory.cc
index a4879386a9a4a6368ca2bee3e4b8ebcaac0a146f..035b8f461300c88195208ca40310ef85c8ef5e9a 100644
--- a/chrome/browser/dom_distiller/dom_distiller_service_factory.cc
+++ b/chrome/browser/dom_distiller/dom_distiller_service_factory.cc
@@ -16,8 +16,12 @@ namespace dom_distiller {
DomDistillerContextKeyedService::DomDistillerContextKeyedService(
scoped_ptr<DomDistillerStoreInterface> store,
- scoped_ptr<DistillerFactory> distiller_factory)
- : DomDistillerService(store.Pass(), distiller_factory.Pass()) {}
+ scoped_ptr<DistillerFactory> distiller_factory,
+ scoped_ptr<DistillerPageFactory> distiller_page_factory)
+ : DomDistillerService(store.Pass(),
+ distiller_factory.Pass(),
+ distiller_page_factory.Pass()) {
+}
// static
DomDistillerServiceFactory* DomDistillerServiceFactory::GetInstance() {
@@ -58,13 +62,14 @@ KeyedService* DomDistillerServiceFactory::BuildServiceInstanceFor(
new DistillerPageWebContentsFactory(profile));
scoped_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory(
new DistillerURLFetcherFactory(profile->GetRequestContext()));
- scoped_ptr<DistillerFactory> distiller_factory(new DistillerFactoryImpl(
- distiller_page_factory.Pass(), distiller_url_fetcher_factory.Pass()));
+ scoped_ptr<DistillerFactory> distiller_factory(
+ new DistillerFactoryImpl(distiller_url_fetcher_factory.Pass()));
DomDistillerContextKeyedService* service =
new DomDistillerContextKeyedService(
dom_distiller_store.PassAs<DomDistillerStoreInterface>(),
- distiller_factory.Pass());
+ distiller_factory.Pass(),
+ distiller_page_factory.Pass());
return service;
}

Powered by Google App Engine
This is Rietveld 408576698