[Reading List iOS] Store distillation date and size.

ios/chrome/browser/reading_list/url_downloader_unittest.mm

 // Copyright 2016 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 "ios/chrome/browser/reading_list/url_downloader.h"
 
 #include <vector>
 
 #include "base/files/file_util.h"
 #import "base/mac/bind_objc_block.h"
@@ skipping 83 matching lines @@
   bool fetching_pdf_;
 
  private:
   void DownloadURL(const GURL& url, bool offline_url_exists) override {
     if (offline_url_exists) {
       DownloadCompletionHandler(url, std::string(), base::FilePath(),
                                 DOWNLOAD_EXISTS);
       return;
     }
     original_url_ = url;
+    saved_size_ = 0;
     distiller_.reset(new DistillerViewerTest(
         url,
         base::Bind(&URLDownloader::DistillerCallback, base::Unretained(this)),
         this, html_, redirect_url_, mime_type_));
   }
 
   void FetchPDFFile() override { fetching_pdf_ = true; }
 
   void OnEndDownload(const GURL& url,
                      const GURL& distilled_url,
                      SuccessState success,
                      const base::FilePath& distilled_path,
+                     int64_t size,
                      const std::string& title) {
     downloaded_files_.push_back(url);
-    DCHECK_EQ(distilled_url, redirect_url_);
+    // Saved data is the string "html".

[gambard, 2017/02/22 12:30:16]

No tests for images?

[Olivier, 2017/02/22 14:05:11]

Added an image "image"

+    EXPECT_EQ(size, 4);
+    EXPECT_EQ(distilled_url, redirect_url_);
   }
 
   void OnEndRemove(const GURL& url, bool success) {
     removed_files_.push_back(url);
   }
 };
 
 namespace {
 class URLDownloaderTest : public testing::Test {
  public:
@@ skipping 123 matching lines @@
   ASSERT_TRUE(std::find(downloader_->downloaded_files_.begin(),
                         downloader_->downloaded_files_.end(),
                         url) != downloader_->downloaded_files_.end());
   ASSERT_TRUE(std::find(downloader_->removed_files_.begin(),
                         downloader_->removed_files_.end(),
                         url) != downloader_->removed_files_.end());
   ASSERT_TRUE(downloader_->CheckExistenceOfOfflineURLPagePath(url));
 }
 
 }  // namespace