Start requiring DistillerPage for calls to DomDistillerService.

components/dom_distiller/core/distiller_unittest.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 <algorithm>
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/values.h"
 #include "components/dom_distiller/core/article_distillation_update.h"
 #include "components/dom_distiller/core/distiller.h"
 #include "components/dom_distiller/core/distiller_page.h"
+#include "components/dom_distiller/core/fake_distiller_page.h"
 #include "components/dom_distiller/core/proto/distilled_article.pb.h"
 #include "components/dom_distiller/core/proto/distilled_page.pb.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using std::vector;
 using std::string;
 using ::testing::Invoke;
 using ::testing::Return;
@@ skipping 144 matching lines @@
       EXPECT_EQ(GetImageName(page_num + 1, img_num),
                 page.image(img_num).name());
     }
   }
 }
 
 }  // namespace
 
 namespace dom_distiller {
 
+using test::MockDistillerPage;
+using test::MockDistillerPageFactory;
+
 class TestDistillerURLFetcher : public DistillerURLFetcher {
  public:
   explicit TestDistillerURLFetcher(bool delay_fetch)
       : DistillerURLFetcher(NULL), delay_fetch_(delay_fetch) {
     responses_[kImageURLs[0]] = string(kImageData[0]);
     responses_[kImageURLs[1]] = string(kImageData[1]);
   }
 
   virtual void FetchURL(const string& url,
                         const URLFetcherCallback& callback) OVERRIDE {
@@ skipping 23 matching lines @@
   TestDistillerURLFetcherFactory() : DistillerURLFetcherFactory(NULL) {}
 
   virtual ~TestDistillerURLFetcherFactory() {}
   virtual DistillerURLFetcher* CreateDistillerURLFetcher() const OVERRIDE {
     return new TestDistillerURLFetcher(false);
   }
 };
 
 class MockDistillerURLFetcherFactory : public DistillerURLFetcherFactory {
  public:
-  MockDistillerURLFetcherFactory()
-      : DistillerURLFetcherFactory(NULL) {}
+  MockDistillerURLFetcherFactory() : DistillerURLFetcherFactory(NULL) {}
   virtual ~MockDistillerURLFetcherFactory() {}
 
   MOCK_CONST_METHOD0(CreateDistillerURLFetcher, DistillerURLFetcher*());
 };
 
-class MockDistillerPage : public DistillerPage {
- public:
-  MOCK_METHOD2(DistillPageImpl, void(const GURL& gurl, const string& script));
-
-  MockDistillerPage() {}
-};
-
-class MockDistillerPageFactory : public DistillerPageFactory {
- public:
-  MOCK_CONST_METHOD0(CreateDistillerPageMock, DistillerPage*());
-
-  virtual scoped_ptr<DistillerPage> CreateDistillerPage() const OVERRIDE {
-    return scoped_ptr<DistillerPage>(CreateDistillerPageMock());
-  }
-};
-
 class DistillerTest : public testing::Test {
  public:
   virtual ~DistillerTest() {}
   void OnDistillArticleDone(scoped_ptr<DistilledArticleProto> proto) {
     article_proto_ = proto.Pass();
   }
 
   void OnDistillArticleUpdate(const ArticleDistillationUpdate& article_update) {
     in_sequence_updates_.push_back(article_update);
   }
 
-  void DistillPage(const std::string& url) {
+  void DistillPage(const std::string& url,
+                   scoped_ptr<DistillerPage> distiller_page) {
     distiller_->DistillPage(GURL(url),
+                            distiller_page.Pass(),
                             base::Bind(&DistillerTest::OnDistillArticleDone,
                                        base::Unretained(this)),
                             base::Bind(&DistillerTest::OnDistillArticleUpdate,
                                        base::Unretained(this)));
   }
 
  protected:
   scoped_ptr<DistillerImpl> distiller_;
   scoped_ptr<DistilledArticleProto> article_proto_;
   std::vector<ArticleDistillationUpdate> in_sequence_updates_;
   MockDistillerPageFactory page_factory_;
   TestDistillerURLFetcherFactory url_fetcher_factory_;
 };
 
 ACTION_P3(DistillerPageOnDistillationDone, distiller_page, url, list) {
   distiller_page->OnDistillationDone(url, list);
 }
 
-ACTION_P2(CreateMockDistillerPage, list, kurl) {
+scoped_ptr<DistillerPage> CreateMockDistillerPage(const base::Value* list,
+                                                  const GURL& url) {
   MockDistillerPage* distiller_page = new MockDistillerPage();
-  EXPECT_CALL(*distiller_page, DistillPageImpl(kurl, _))
-      .WillOnce(
-          DistillerPageOnDistillationDone(distiller_page, kurl, list));
-  return distiller_page;
+  EXPECT_CALL(*distiller_page, DistillPageImpl(url, _))
+      .WillOnce(DistillerPageOnDistillationDone(distiller_page, url, list));
+  return scoped_ptr<DistillerPage>(distiller_page).Pass();
 }
 
-ACTION_P2(CreateMockDistillerPageWithPendingJSCallback,
-          distiller_page_ptr,
-          kurl) {
+scoped_ptr<DistillerPage> CreateMockDistillerPageWithPendingJSCallback(
+    MockDistillerPage** distiller_page_ptr,
+    const GURL& url) {
   MockDistillerPage* distiller_page = new MockDistillerPage();
   *distiller_page_ptr = distiller_page;
-  EXPECT_CALL(*distiller_page, DistillPageImpl(kurl, _));
-  return distiller_page;
+  EXPECT_CALL(*distiller_page, DistillPageImpl(url, _));
+  return scoped_ptr<DistillerPage>(distiller_page).Pass();
 }
 
-ACTION_P3(CreateMockDistillerPages,
-          distiller_data,
-          pages_size,
-          start_page_num) {
+scoped_ptr<DistillerPage> CreateMockDistillerPages(
+    MultipageDistillerData* distiller_data,
+    size_t pages_size,
+    int start_page_num) {
   MockDistillerPage* distiller_page = new MockDistillerPage();
   {
     testing::InSequence s;
     vector<int> page_nums = GetPagesInSequence(start_page_num, pages_size);
     for (size_t page_num = 0; page_num < pages_size; ++page_num) {
       int page = page_nums[page_num];
       GURL url = GURL(distiller_data->page_urls[page]);
       EXPECT_CALL(*distiller_page, DistillPageImpl(url, _))
           .WillOnce(DistillerPageOnDistillationDone(
               distiller_page, url, distiller_data->distilled_values[page]));
     }
   }
-  return distiller_page;
+  return scoped_ptr<DistillerPage>(distiller_page).Pass();
 }
 
 TEST_F(DistillerTest, DistillPage) {
   base::MessageLoopForUI loop;
   scoped_ptr<base::ListValue> list =
       CreateDistilledValueReturnedFromJS(kTitle, kContent, vector<int>(), "");
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPage(list.get(), GURL(kURL)));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(kURL);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(kURL, CreateMockDistillerPage(list.get(), GURL(kURL)).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kTitle, article_proto_->title());
   EXPECT_EQ(article_proto_->pages_size(), 1);
   const DistilledPageProto& first_page = article_proto_->pages(0);
   EXPECT_EQ(kContent, first_page.html());
   EXPECT_EQ(kURL, first_page.url());
 }
 
 TEST_F(DistillerTest, DistillPageWithImages) {
   base::MessageLoopForUI loop;
   vector<int> image_indices;
   image_indices.push_back(0);
   image_indices.push_back(1);
   scoped_ptr<base::ListValue> list =
       CreateDistilledValueReturnedFromJS(kTitle, kContent, image_indices, "");
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPage(list.get(), GURL(kURL)));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(kURL);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(kURL, CreateMockDistillerPage(list.get(), GURL(kURL)).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kTitle, article_proto_->title());
   EXPECT_EQ(article_proto_->pages_size(), 1);
   const DistilledPageProto& first_page = article_proto_->pages(0);
   EXPECT_EQ(kContent, first_page.html());
   EXPECT_EQ(kURL, first_page.url());
   EXPECT_EQ(2, first_page.image_size());
   EXPECT_EQ(kImageData[0], first_page.image(0).data());
   EXPECT_EQ(GetImageName(1, 0), first_page.image(0).name());
   EXPECT_EQ(kImageData[1], first_page.image(1).data());
@@ skipping 12 matching lines @@
     // Each page has different number of images.
     size_t tot_images = (page_num + kTotalImages) % (kTotalImages + 1);
     vector<int> image_indices;
     for (size_t img_num = 0; img_num < tot_images; img_num++) {
       image_indices.push_back(next_image_number);
       next_image_number = (next_image_number + 1) % kTotalImages;
     }
     distiller_data->image_ids.push_back(image_indices);
   }
 
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPages(distiller_data.get(), kNumPages, 0));
-
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(distiller_data->page_urls[0]);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(
+      distiller_data->page_urls[0],
+      CreateMockDistillerPages(distiller_data.get(), kNumPages, 0).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   VerifyArticleProtoMatchesMultipageData(
       article_proto_.get(), distiller_data.get(), kNumPages);
 }
 
 TEST_F(DistillerTest, DistillLinkLoop) {
   base::MessageLoopForUI loop;
   // Create a loop, the next page is same as the current page. This could
   // happen if javascript misparses a next page link.
   scoped_ptr<base::ListValue> list =
       CreateDistilledValueReturnedFromJS(kTitle, kContent, vector<int>(), kURL);
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPage(list.get(), GURL(kURL)));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(kURL);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(kURL, CreateMockDistillerPage(list.get(), GURL(kURL)).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kTitle, article_proto_->title());
   EXPECT_EQ(article_proto_->pages_size(), 1);
 }
 
 TEST_F(DistillerTest, CheckMaxPageLimitExtraPage) {
   base::MessageLoopForUI loop;
   const size_t kMaxPagesInArticle = 10;
   scoped_ptr<MultipageDistillerData> distiller_data =
       CreateMultipageDistillerDataWithoutImages(kMaxPagesInArticle);
 
   // Note: Next page url of the last page of article is set. So distiller will
   // try to do kMaxPagesInArticle + 1 calls if the max article limit does not
   // work.
   scoped_ptr<base::ListValue> last_page_data =
       CreateDistilledValueReturnedFromJS(
           kTitle,
           distiller_data->content[kMaxPagesInArticle - 1],
           vector<int>(),
           "",
           distiller_data->page_urls[kMaxPagesInArticle - 2]);
 
   distiller_data->distilled_values.pop_back();
   distiller_data->distilled_values.push_back(last_page_data.release());
 
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock()).WillOnce(
-      CreateMockDistillerPages(distiller_data.get(), kMaxPagesInArticle, 0));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
 
   distiller_->SetMaxNumPagesInArticle(kMaxPagesInArticle);
 
-  DistillPage(distiller_data->page_urls[0]);
+  DistillPage(distiller_data->page_urls[0],
+              CreateMockDistillerPages(
+                  distiller_data.get(), kMaxPagesInArticle, 0).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kTitle, article_proto_->title());
   EXPECT_EQ(kMaxPagesInArticle,
             static_cast<size_t>(article_proto_->pages_size()));
 }
 
 TEST_F(DistillerTest, CheckMaxPageLimitExactLimit) {
   base::MessageLoopForUI loop;
   const size_t kMaxPagesInArticle = 10;
   scoped_ptr<MultipageDistillerData> distiller_data =
       CreateMultipageDistillerDataWithoutImages(kMaxPagesInArticle);
 
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock()).WillOnce(
-      CreateMockDistillerPages(distiller_data.get(), kMaxPagesInArticle, 0));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
 
   // Check if distilling an article with exactly the page limit works.
   distiller_->SetMaxNumPagesInArticle(kMaxPagesInArticle);
 
-  DistillPage(distiller_data->page_urls[0]);
+  DistillPage(distiller_data->page_urls[0],
+              CreateMockDistillerPages(
+                  distiller_data.get(), kMaxPagesInArticle, 0).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kTitle, article_proto_->title());
   EXPECT_EQ(kMaxPagesInArticle,
             static_cast<size_t>(article_proto_->pages_size()));
 }
 
 TEST_F(DistillerTest, SinglePageDistillationFailure) {
   base::MessageLoopForUI loop;
   // To simulate failure return a null value.
   scoped_ptr<base::Value> nullValue(base::Value::CreateNullValue());
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPage(nullValue.get(), GURL(kURL)));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(kURL);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(kURL,
+              CreateMockDistillerPage(nullValue.get(), GURL(kURL)).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ("", article_proto_->title());
   EXPECT_EQ(0, article_proto_->pages_size());
 }
 
 TEST_F(DistillerTest, MultiplePagesDistillationFailure) {
   base::MessageLoopForUI loop;
   const size_t kNumPages = 8;
   scoped_ptr<MultipageDistillerData> distiller_data =
       CreateMultipageDistillerDataWithoutImages(kNumPages);
 
   // The page number of the failed page.
   size_t failed_page_num = 3;
   // reset distilled data of the failed page.
   distiller_data->distilled_values.erase(
       distiller_data->distilled_values.begin() + failed_page_num);
   distiller_data->distilled_values.insert(
       distiller_data->distilled_values.begin() + failed_page_num,
       base::Value::CreateNullValue());
   // Expect only calls till the failed page number.
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock()).WillOnce(
-      CreateMockDistillerPages(distiller_data.get(), failed_page_num + 1, 0));
-
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(distiller_data->page_urls[0]);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(distiller_data->page_urls[0],
+              CreateMockDistillerPages(
+                  distiller_data.get(), failed_page_num + 1, 0).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kTitle, article_proto_->title());
   VerifyArticleProtoMatchesMultipageData(
       article_proto_.get(), distiller_data.get(), failed_page_num);
 }
 
 TEST_F(DistillerTest, DistillPreviousPage) {
   base::MessageLoopForUI loop;
   const size_t kNumPages = 8;
 
   // The page number of the article on which distillation starts.
   int start_page_num = 3;
   scoped_ptr<MultipageDistillerData> distiller_data =
       CreateMultipageDistillerDataWithoutImages(kNumPages);
 
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPages(
-          distiller_data.get(), kNumPages, start_page_num));
-
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(distiller_data->page_urls[start_page_num]);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(distiller_data->page_urls[start_page_num],
+              CreateMockDistillerPages(
+                  distiller_data.get(), kNumPages, start_page_num).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   VerifyArticleProtoMatchesMultipageData(
       article_proto_.get(), distiller_data.get(), kNumPages);
 }
 
 TEST_F(DistillerTest, IncrementalUpdates) {
   base::MessageLoopForUI loop;
   const size_t kNumPages = 8;
 
   // The page number of the article on which distillation starts.
   int start_page_num = 3;
   scoped_ptr<MultipageDistillerData> distiller_data =
       CreateMultipageDistillerDataWithoutImages(kNumPages);
 
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPages(
-          distiller_data.get(), kNumPages, start_page_num));
-
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(distiller_data->page_urls[start_page_num]);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(distiller_data->page_urls[start_page_num],
+              CreateMockDistillerPages(
+                  distiller_data.get(), kNumPages, start_page_num).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kTitle, article_proto_->title());
   EXPECT_EQ(kNumPages, static_cast<size_t>(article_proto_->pages_size()));
   EXPECT_EQ(kNumPages, in_sequence_updates_.size());
 
   VerifyIncrementalUpdatesMatch(
       distiller_data.get(), kNumPages, in_sequence_updates_, start_page_num);
 }
 
 TEST_F(DistillerTest, IncrementalUpdatesDoNotDeleteFinalArticle) {
   base::MessageLoopForUI loop;
   const size_t kNumPages = 8;
   int start_page_num = 3;
   scoped_ptr<MultipageDistillerData> distiller_data =
       CreateMultipageDistillerDataWithoutImages(kNumPages);
 
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPages(
-          distiller_data.get(), kNumPages, start_page_num));
-
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(distiller_data->page_urls[start_page_num]);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(distiller_data->page_urls[start_page_num],
+              CreateMockDistillerPages(
+                  distiller_data.get(), kNumPages, start_page_num).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kNumPages, in_sequence_updates_.size());
 
   in_sequence_updates_.clear();
 
   // Should still be able to access article and pages.
   VerifyArticleProtoMatchesMultipageData(
       article_proto_.get(), distiller_data.get(), kNumPages);
 }
 
 TEST_F(DistillerTest, DeletingArticleDoesNotInterfereWithUpdates) {
   base::MessageLoopForUI loop;
   const size_t kNumPages = 8;
   scoped_ptr<MultipageDistillerData> distiller_data =
       CreateMultipageDistillerDataWithoutImages(kNumPages);
   // The page number of the article on which distillation starts.
   int start_page_num = 3;
 
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPages(
-          distiller_data.get(), kNumPages, start_page_num));
-
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(distiller_data->page_urls[start_page_num]);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(distiller_data->page_urls[start_page_num],
+              CreateMockDistillerPages(
+                  distiller_data.get(), kNumPages, start_page_num).Pass());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(kNumPages, in_sequence_updates_.size());
   EXPECT_EQ(kTitle, article_proto_->title());
   EXPECT_EQ(kNumPages, static_cast<size_t>(article_proto_->pages_size()));
 
   // Delete the article.
   article_proto_.reset();
   VerifyIncrementalUpdatesMatch(
       distiller_data.get(), kNumPages, in_sequence_updates_, start_page_num);
 }
 
 TEST_F(DistillerTest, CancelWithDelayedImageFetchCallback) {
   base::MessageLoopForUI loop;
   vector<int> image_indices;
   image_indices.push_back(0);
   scoped_ptr<base::ListValue> distilled_value =
       CreateDistilledValueReturnedFromJS(kTitle, kContent, image_indices, "");
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPage(distilled_value.get(), GURL(kURL)));
-
   TestDistillerURLFetcher* delayed_fetcher = new TestDistillerURLFetcher(true);
   MockDistillerURLFetcherFactory url_fetcher_factory;
   EXPECT_CALL(url_fetcher_factory, CreateDistillerURLFetcher())
       .WillOnce(Return(delayed_fetcher));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory));
-  DistillPage(kURL);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory));
+  DistillPage(
+      kURL, CreateMockDistillerPage(distilled_value.get(), GURL(kURL)).Pass());
   base::MessageLoop::current()->RunUntilIdle();
 
   // Post callback from the url fetcher and then delete the distiller.
   delayed_fetcher->PostCallbackTask();
   distiller_.reset();
 
   base::MessageLoop::current()->RunUntilIdle();
 }
 
 TEST_F(DistillerTest, CancelWithDelayedJSCallback) {
   base::MessageLoopForUI loop;
   scoped_ptr<base::ListValue> distilled_value =
       CreateDistilledValueReturnedFromJS(kTitle, kContent, vector<int>(), "");
   MockDistillerPage* distiller_page = NULL;
-  EXPECT_CALL(page_factory_, CreateDistillerPageMock())
-      .WillOnce(CreateMockDistillerPageWithPendingJSCallback(&distiller_page,
-                                                             GURL(kURL)));
-  distiller_.reset(new DistillerImpl(page_factory_, url_fetcher_factory_));
-  DistillPage(kURL);
+  distiller_.reset(new DistillerImpl(url_fetcher_factory_));
+  DistillPage(kURL,
+              CreateMockDistillerPageWithPendingJSCallback(&distiller_page,
+                                                           GURL(kURL)));
   base::MessageLoop::current()->RunUntilIdle();
 
   ASSERT_TRUE(distiller_page);
   // Post the task to execute javascript and then delete the distiller.
   distiller_page->OnDistillationDone(GURL(kURL), distilled_value.get());
   distiller_.reset();
 
   base::MessageLoop::current()->RunUntilIdle();
 }
 
 }  // namespace dom_distiller