| Index: components/dom_distiller/standalone/content_extractor_browsertest.cc
|
| diff --git a/components/dom_distiller/standalone/content_extractor_browsertest.cc b/components/dom_distiller/standalone/content_extractor_browsertest.cc
|
| index 3655826e0de358503fcbcd596d74888d642cea10..d0d856d8b730eb632c7d988154bef93ab7141a2b 100644
|
| --- a/components/dom_distiller/standalone/content_extractor_browsertest.cc
|
| +++ b/components/dom_distiller/standalone/content_extractor_browsertest.cc
|
| @@ -56,7 +56,7 @@ typedef base::hash_map<std::string, std::string> FileToUrlMap;
|
| class TestDistillerFactoryImpl : public DistillerFactory {
|
| public:
|
| TestDistillerFactoryImpl(
|
| - scoped_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory,
|
| + std::unique_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory,
|
| const dom_distiller::proto::DomDistillerOptions& dom_distiller_options,
|
| const FileToUrlMap& file_to_url_map)
|
| : distiller_url_fetcher_factory_(
|
| @@ -66,20 +66,20 @@ class TestDistillerFactoryImpl : public DistillerFactory {
|
|
|
| ~TestDistillerFactoryImpl() override {}
|
|
|
| - scoped_ptr<Distiller> CreateDistillerForUrl(const GURL& url) override {
|
| + std::unique_ptr<Distiller> CreateDistillerForUrl(const GURL& url) override {
|
| dom_distiller::proto::DomDistillerOptions options;
|
| options = dom_distiller_options_;
|
| FileToUrlMap::const_iterator it = file_to_url_map_.find(url.spec());
|
| if (it != file_to_url_map_.end()) {
|
| options.set_original_url(it->second);
|
| }
|
| - scoped_ptr<DistillerImpl> distiller(new DistillerImpl(
|
| - *distiller_url_fetcher_factory_, options));
|
| + std::unique_ptr<DistillerImpl> distiller(
|
| + new DistillerImpl(*distiller_url_fetcher_factory_, options));
|
| return std::move(distiller);
|
| }
|
|
|
| private:
|
| - scoped_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory_;
|
| + std::unique_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory_;
|
| dom_distiller::proto::DomDistillerOptions dom_distiller_options_;
|
| FileToUrlMap file_to_url_map_;
|
| };
|
| @@ -121,7 +121,7 @@ const char* kPaginationAlgo = "pagination-algo";
|
| // Maximum number of concurrent started extractor requests.
|
| const int kMaxExtractorTasks = 8;
|
|
|
| -scoped_ptr<DomDistillerService> CreateDomDistillerService(
|
| +std::unique_ptr<DomDistillerService> CreateDomDistillerService(
|
| content::BrowserContext* context,
|
| const base::FilePath& db_path,
|
| const FileToUrlMap& file_to_url_map) {
|
| @@ -131,18 +131,18 @@ scoped_ptr<DomDistillerService> CreateDomDistillerService(
|
|
|
| // TODO(cjhopman): use an in-memory database instead of an on-disk one with
|
| // temporary directory.
|
| - scoped_ptr<leveldb_proto::ProtoDatabaseImpl<ArticleEntry> > db(
|
| + std::unique_ptr<leveldb_proto::ProtoDatabaseImpl<ArticleEntry>> db(
|
| new leveldb_proto::ProtoDatabaseImpl<ArticleEntry>(
|
| background_task_runner));
|
| - scoped_ptr<DomDistillerStore> dom_distiller_store(
|
| + std::unique_ptr<DomDistillerStore> dom_distiller_store(
|
| new DomDistillerStore(std::move(db), db_path));
|
|
|
| - scoped_ptr<DistillerPageFactory> distiller_page_factory(
|
| + std::unique_ptr<DistillerPageFactory> distiller_page_factory(
|
| new DistillerPageWebContentsFactory(context));
|
| - scoped_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory(
|
| + std::unique_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory(
|
| new DistillerURLFetcherFactory(
|
| - content::BrowserContext::GetDefaultStoragePartition(context)->
|
| - GetURLRequestContext()));
|
| + content::BrowserContext::GetDefaultStoragePartition(context)
|
| + ->GetURLRequestContext()));
|
|
|
| dom_distiller::proto::DomDistillerOptions options;
|
| if (base::CommandLine::ForCurrentProcess()->HasSwitch(kExtractTextOnly)) {
|
| @@ -166,18 +166,20 @@ scoped_ptr<DomDistillerService> CreateDomDistillerService(
|
| kPaginationAlgo));
|
| }
|
|
|
| - scoped_ptr<DistillerFactory> distiller_factory(new TestDistillerFactoryImpl(
|
| - std::move(distiller_url_fetcher_factory), options, file_to_url_map));
|
| + std::unique_ptr<DistillerFactory> distiller_factory(
|
| + new TestDistillerFactoryImpl(std::move(distiller_url_fetcher_factory),
|
| + options, file_to_url_map));
|
|
|
| // Setting up PrefService for DistilledPagePrefs.
|
| user_prefs::TestingPrefServiceSyncable* pref_service =
|
| new user_prefs::TestingPrefServiceSyncable();
|
| DistilledPagePrefs::RegisterProfilePrefs(pref_service->registry());
|
|
|
| - return scoped_ptr<DomDistillerService>(new DomDistillerService(
|
| + return std::unique_ptr<DomDistillerService>(new DomDistillerService(
|
| std::move(dom_distiller_store), std::move(distiller_factory),
|
| std::move(distiller_page_factory),
|
| - scoped_ptr<DistilledPagePrefs>(new DistilledPagePrefs(pref_service))));
|
| + std::unique_ptr<DistilledPagePrefs>(
|
| + new DistilledPagePrefs(pref_service))));
|
| }
|
|
|
| void AddComponentsTestResources() {
|
| @@ -311,7 +313,7 @@ class ContentExtractionRequest : public ViewRequestDelegate {
|
| }
|
|
|
| const DistilledArticleProto* article_proto_;
|
| - scoped_ptr<ViewerHandle> viewer_handle_;
|
| + std::unique_ptr<ViewerHandle> viewer_handle_;
|
| GURL url_;
|
| base::Closure finished_callback_;
|
| };
|
| @@ -427,12 +429,14 @@ class ContentExtractor : public ContentBrowserTest {
|
| size_t next_request_;
|
|
|
| base::ScopedTempDir db_dir_;
|
| - scoped_ptr<net::ScopedDefaultHostResolverProc> mock_host_resolver_override_;
|
| - scoped_ptr<DomDistillerService> service_;
|
| + std::unique_ptr<net::ScopedDefaultHostResolverProc>
|
| + mock_host_resolver_override_;
|
| + std::unique_ptr<DomDistillerService> service_;
|
| ScopedVector<ContentExtractionRequest> requests_;
|
|
|
| std::string output_data_;
|
| - scoped_ptr<google::protobuf::io::StringOutputStream> protobuf_output_stream_;
|
| + std::unique_ptr<google::protobuf::io::StringOutputStream>
|
| + protobuf_output_stream_;
|
| };
|
|
|
| IN_PROC_BROWSER_TEST_F(ContentExtractor, MANUAL_ExtractUrl) {
|
|
|