add options to specify original domain(s)

components/dom_distiller/core/distiller.cc

 // Copyright 2013 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 "components/dom_distiller/core/distiller.h"
 
 #include <map>
 #include <vector>
 
 #include "base/auto_reset.h"
@@ skipping 19 matching lines @@
 
 DistillerFactoryImpl::DistillerFactoryImpl(
     scoped_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory,
     const dom_distiller::proto::DomDistillerOptions& dom_distiller_options)
     : distiller_url_fetcher_factory_(distiller_url_fetcher_factory.Pass()),
       dom_distiller_options_(dom_distiller_options) {
 }
 
 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();
 }
 
 DistillerImpl::DistilledPageData::DistilledPageData() {}
 
 DistillerImpl::DistilledPageData::~DistilledPageData() {}
 
 DistillerImpl::DistillerImpl(
@@ skipping 273 matching lines @@
     DCHECK(finished_pages_index_.empty());
 
     base::AutoReset<bool> dont_delete_this_in_callback(&destruction_allowed_,
                                                        false);
     finished_cb_.Run(article_proto.Pass());
     finished_cb_.Reset();
   }
 }
 
 }  // namespace dom_distiller