Implement GetJobResponse and merge JobData into DataManager.

content/public/test/fake_download_item.cc

Index: content/public/test/fake_download_item.cc
diff --git a/content/public/test/fake_download_item.cc b/content/public/test/fake_download_item.cc
index 4bb1e706a32fb2985eba9f1d06c30c4d19423b5d..06b12607d4857dab10300754c741bbb38390c88e 100644
--- a/content/public/test/fake_download_item.cc
+++ b/content/public/test/fake_download_item.cc
@@ -134,6 +134,14 @@ const GURL& FakeDownloadItem::GetOriginalUrl() const {
   return original_url_;
 }
 
+void FakeDownloadItem::SetReceivedBytes(int64_t received_bytes) {

[Peter Beverloo, 2017/03/25 03:38:50]

Should this trigger NotifyDownloadUpdated()?

[harkness, 2017/03/26 16:13:18]

It could, but that would break the current model (see SetState()) that the test
author has to make 2 calls: 1 to update the DownloadItem and then 1 to
explicitly invoke the method on the Observer.

I wouldn't mind changing it to use that approach, but we'd need to change it
everywhere to be consistent.

+  received_bytes_ = received_bytes;
+}
+
+int64_t FakeDownloadItem::GetReceivedBytes() const {
+  return received_bytes_;
+}
+
 // The methods below are not supported and are not expected to be called.
 void FakeDownloadItem::ValidateDangerousDownload() {
   NOTREACHED();
@@ -325,11 +333,6 @@ int64_t FakeDownloadItem::GetTotalBytes() const {
   return 1;
 }
 
-int64_t FakeDownloadItem::GetReceivedBytes() const {
-  NOTREACHED();
-  return 1;
-}
-
 const std::vector<DownloadItem::ReceivedSlice>&
 FakeDownloadItem::GetReceivedSlices() const {
   NOTREACHED();