Introduce ParallelDownloadJob.

content/browser/download/download_job.cc

 // Copyright 2017 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 "content/browser/download/download_job.h"
 
+#include "content/browser/download/download_item_impl.h"
+
 namespace content {
 
-// Unknown download progress.
-const int kDownloadProgressUnknown = -1;
-
-// Unknown download speed.
-const int kDownloadSpeedUnknown = -1;
-
-DownloadJob::DownloadJob() : manager_(nullptr) {}
+DownloadJob::DownloadJob() : download_item_(nullptr) {}
 
 DownloadJob::~DownloadJob() = default;
 
-void DownloadJob::OnAttached(DownloadJob::Manager* manager) {
-  DCHECK(!manager_) << "DownloadJob::Manager has already been attached.";
-  manager_ = manager;
+void DownloadJob::Attach(DownloadItemImpl* download_item) {
+  download_item_ = download_item;
 }
 
-void DownloadJob::OnBeforeDetach() {
-  manager_ = nullptr;
-}
-
-void DownloadJob::StartedSavingResponse() {
-  if (manager_)
-    manager_->OnSavingStarted(this);
-}
-
-void DownloadJob::Interrupt(DownloadInterruptReason reason) {
-  if (manager_)
-    manager_->OnDownloadInterrupted(this, reason);
-}
-
-void DownloadJob::Complete() {
-  if (manager_)
-    manager_->OnDownloadComplete(this);
+void DownloadJob::InitializeDownloadFile() const {
+  download_item_->InitializeDownloadFile();

[asanka, 2017/02/16 16:27:39]

It would be great if we could have calls going one way. I.e. does it make sense
for DownloadItemImpl to retain ownership of the DownloadFile? Or should
DownloadJob be the owner of the DownloadFile.

DownloadFile is only supposed to contain state for an active download, which
maps to the lifetime of a Job.

[xingliu, 2017/02/20 18:59:10]

Ideally the file should live with DownloadJob subclass, can we do it in a
separate CL, since this will involve a lot of refactoring.

Also I'm fine with adding the parallel logic in DownloadItemImpl, maybe adding
all these DownloadJob classes are over abstraction for parallel download, wdyt?

 }
 
 }  // namespace content