Add the JobComplete callback and error/interrupt information

content/public/test/fake_download_item.h

Index: content/public/test/fake_download_item.h
diff --git a/content/public/test/fake_download_item.h b/content/public/test/fake_download_item.h
index e11054a86173d9fa3c002bd0952b56f3d5f32061..198a38656675ab6462ed7ba8cce858a4ac69d6a1 100644
--- a/content/public/test/fake_download_item.h
+++ b/content/public/test/fake_download_item.h
@@ -66,6 +66,9 @@ class FakeDownloadItem : public DownloadItem {
   void SetOriginalUrl(const GURL& url);
   const GURL& GetOriginalUrl() const override;
 
+  void SetLastReason(DownloadInterruptReason);
+  DownloadInterruptReason GetLastReason() const override;
+
   // The methods below are not supported and are not expected to be called.
   void ValidateDangerousDownload() override;
   void StealDangerousDownload(bool delete_file_afterward,
@@ -76,7 +79,6 @@ class FakeDownloadItem : public DownloadItem {
   void Remove() override;
   void OpenDownload() override;
   void ShowDownloadInShell() override;
-  DownloadInterruptReason GetLastReason() const override;
   bool IsPaused() const override;
   bool IsTemporary() const override;
   bool CanResume() const override;
@@ -139,6 +141,7 @@ class FakeDownloadItem : public DownloadItem {
   DownloadState download_state_;
   std::string mime_type_;
   GURL original_url_;
+  DownloadInterruptReason last_reason_;

[Peter Beverloo, 2017/03/15 16:53:38]

This is a scalar, we need to initialize it. |id_| and
|is_file_externally_removed_| and |download_state_| too...

[harkness, 2017/03/16 11:41:10]

Done.

 
   // The members below are to be returned by methods, which return by reference.
   std::string dummy_string;