Give the BackgroundFetchJobController a simple state.

content/browser/background_fetch/background_fetch_context.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_context.h"
 
 #include "base/memory/ptr_util.h"
 #include "content/browser/background_fetch/background_fetch_data_manager.h"
 #include "content/browser/background_fetch/background_fetch_job_controller.h"
 #include "content/browser/background_fetch/background_fetch_registration_id.h"
@@ skipping 89 matching lines @@
 }
 
 BackgroundFetchJobController* BackgroundFetchContext::GetActiveFetch(
     const BackgroundFetchRegistrationId& registration_id) const {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   auto iter = active_fetches_.find(registration_id);
   if (iter == active_fetches_.end())
     return nullptr;
 
-  return iter->second.get();
+  BackgroundFetchJobController* controller = iter->second.get();
+  if (controller->state() == BackgroundFetchJobController::State::ABORTED ||
+      controller->state() == BackgroundFetchJobController::State::COMPLETED) {
+    return nullptr;
+  }
+
+  return controller;
 }
 
 void BackgroundFetchContext::CreateController(
     const BackgroundFetchRegistrationId& registration_id,
     const BackgroundFetchOptions& options) {
   std::unique_ptr<BackgroundFetchJobController> controller =
       base::MakeUnique<BackgroundFetchJobController>(
           registration_id, options, browser_context_, storage_partition_,
           background_fetch_data_manager_.get(),
-          base::BindOnce(&BackgroundFetchContext::DidFinishFetch, this));
+          base::BindOnce(&BackgroundFetchContext::DidCompleteJob, this));
 
   active_fetches_.insert(
       std::make_pair(registration_id, std::move(controller)));
 }
 
-void BackgroundFetchContext::DidFinishFetch(
-    const BackgroundFetchRegistrationId& registration_id,
-    bool aborted_by_developer) {
+void BackgroundFetchContext::DidCompleteJob(
+    BackgroundFetchJobController* controller) {
+  const BackgroundFetchRegistrationId& registration_id =
+      controller->registration_id();
+
   DCHECK_GT(active_fetches_.count(registration_id), 0u);
 
-  // TODO(peter): Dispatch the `backgroundfetched` or the `backgroundfetchfail`
-  // event to the Service Worker when |aborted_by_developer| is not set.
+  if (controller->state() == BackgroundFetchJobController::State::COMPLETED) {
+    // TODO(peter): Dispatch the `backgroundfetched` or `backgroundfetchfail`
+    // event to the Service Worker to inform the developer.
+  }
 
   active_fetches_.erase(registration_id);
 }
 
 }  // namespace content