Give the BackgroundFetchJobController a simple state.

content/browser/background_fetch/background_fetch_job_controller.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/background_fetch/background_fetch_job_controller.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/guid.h"
@@ skipping 39 matching lines @@
   download_guid_map_.clear();
 
   // TODO(harkness): Write final status to the DataManager. After this call, the
   // |data_manager_| is no longer valid.
   data_manager_ = nullptr;
 }
 
 void BackgroundFetchJobController::StartProcessing() {
   DCHECK(data_manager_);
 
+  state_ = State::FETCHING;
+
   const BackgroundFetchRequestInfo& fetch_request =
       data_manager_->GetNextBackgroundFetchRequestInfo(job_guid_);
   ProcessRequest(fetch_request);
 
   // Currently, this processes a single request at a time.
 }
 
 void BackgroundFetchJobController::UpdateUI(const std::string& title) {
   // TODO(harkness): Update the user interface with |title|.
 }
 
 void BackgroundFetchJobController::Abort() {
   // TODO(harkness): Abort all in-progress downloads.
 
-  std::move(completed_callback_)
-      .Run(registration_id_, true /* aborted_by_developer */);
+  state_ = State::ABORTED;
+
+  std::move(completed_callback_).Run(this);
 }
 
 void BackgroundFetchJobController::DownloadStarted(
     const std::string& request_guid,
     DownloadItem* item,
     DownloadInterruptReason reason) {
   // Start observing the DownloadItem. No need to store the pointer because it
   // can be retrieved from the DownloadManager.
   item->AddObserver(this);
   download_guid_map_[item->GetGuid()] = request_guid;
@@ skipping 21 matching lines @@
                                                item->GetReceivedBytes());
     // Fall through.
     case DownloadItem::DownloadState::CANCELLED:
       // TODO(harkness): Tell the notification service to update the download
       // notification.
 
       if (requests_remaining) {
         ProcessRequest(
             data_manager_->GetNextBackgroundFetchRequestInfo(job_guid_));
       } else if (data_manager_->IsComplete(job_guid_)) {
-        std::move(completed_callback_)
-            .Run(registration_id_, false /* aborted_by_developer */);
+        state_ = State::COMPLETED;
+        std::move(completed_callback_).Run(this);
       }
       break;
     case DownloadItem::DownloadState::INTERRUPTED:
       // TODO(harkness): Just update the notification that it is paused.
       break;
     case DownloadItem::DownloadState::IN_PROGRESS:
       // TODO(harkness): If the download was previously paused, this should now
       // unpause the notification.
       break;
     case DownloadItem::DownloadState::MAX_DOWNLOAD_STATE:
@@ skipping 18 matching lines @@
           fetch_request.GetURL(), storage_partition_->GetURLRequestContext()));
   params->set_callback(
       base::Bind(&BackgroundFetchJobController::DownloadStarted,
                  weak_ptr_factory_.GetWeakPtr(), fetch_request.guid()));
 
   BrowserContext::GetDownloadManager(browser_context_)
       ->DownloadUrl(std::move(params));
 }
 
 }  // namespace content