add options to specify original domain(s)

components/dom_distiller/core/distiller.cc

Index: components/dom_distiller/core/distiller.cc
diff --git a/components/dom_distiller/core/distiller.cc b/components/dom_distiller/core/distiller.cc
index b5bd95dab38ade55ca266fccb64b5ea1e5cb8fc2..c67390dcdf797decda9eb226f36470fe59e73328 100644
--- a/components/dom_distiller/core/distiller.cc
+++ b/components/dom_distiller/core/distiller.cc
@@ -37,7 +37,12 @@ DistillerFactoryImpl::DistillerFactoryImpl(
 
 DistillerFactoryImpl::~DistillerFactoryImpl() {}
 
-scoped_ptr<Distiller> DistillerFactoryImpl::CreateDistiller() {
+scoped_ptr<Distiller> DistillerFactoryImpl::CreateDistillerForUrl(
+    const GURL& unused) {
+  // The |unused| GURL is not needed here or now; it'll be specified later for

[cjhopman, 2014/10/29 17:13:41]

I don't think the comment is necessary. I'd just drop it.

Otherwise, maybe just

// This default implementation has the same behavior for all URLs.

[kuan, 2014/10/29 17:26:53]

Done.

+  // DistillerPage::DistillPage() via TaskTracker::StartDistiller().  It's only
+  // needed if different URLs require different DomDistillerOption's, like the
+  // standalone ContentExtractor.
   scoped_ptr<DistillerImpl> distiller(new DistillerImpl(
       *distiller_url_fetcher_factory_, dom_distiller_options_));
   return distiller.Pass();