Fire the `backgroundfetchfail` event for failed fetches

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_event_dispatcher.h"
 #include "content/browser/background_fetch/background_fetch_job_controller.h"
@@ skipping 175 matching lines @@
   // Get the sequence of settled fetches from the data manager.
   data_manager_->GetSettledFetchesForRegistration(
       registration_id,
       base::BindOnce(&BackgroundFetchContext::DidGetSettledFetches, this,
                      registration_id));
 }
 
 void BackgroundFetchContext::DidGetSettledFetches(
     const BackgroundFetchRegistrationId& registration_id,
     blink::mojom::BackgroundFetchError error,
+    bool background_fetch_succeeded,
     std::vector<BackgroundFetchSettledFetch> settled_fetches,
     std::vector<std::unique_ptr<BlobHandle>> blob_handles) {
   if (error != blink::mojom::BackgroundFetchError::NONE) {
     DeleteRegistration(registration_id, std::move(blob_handles));
     return;
   }
 
-  // TODO(peter): Distinguish between the `backgroundfetched` and
-  // `backgroundfetchfail` events based on the status code of all fetches. We
-  // don't populate that field yet, so always assume it's successful for now.
-
-  event_dispatcher_->DispatchBackgroundFetchedEvent(
-      registration_id, std::move(settled_fetches),
-      base::Bind(&BackgroundFetchContext::DeleteRegistration, this,
-                 registration_id, std::move(blob_handles)));
+  // The `backgroundfetched` event will be invoked when all requests in the
+  // registration have completed successfully. In all other cases, the
+  // `backgroundfetchfail` event will be invoked instead.
+  if (background_fetch_succeeded) {
+    event_dispatcher_->DispatchBackgroundFetchedEvent(
+        registration_id, std::move(settled_fetches),
+        base::Bind(&BackgroundFetchContext::DeleteRegistration, this,
+                   registration_id, std::move(blob_handles)));
+  } else {
+    event_dispatcher_->DispatchBackgroundFetchFailEvent(
+        registration_id, std::move(settled_fetches),
+        base::Bind(&BackgroundFetchContext::DeleteRegistration, this,
+                   registration_id, std::move(blob_handles)));
+  }
 }
 
 void BackgroundFetchContext::DeleteRegistration(
     const BackgroundFetchRegistrationId& registration_id,
     const std::vector<std::unique_ptr<BlobHandle>>& blob_handles) {
   DCHECK_GT(active_fetches_.count(registration_id), 0u);
 
   // Delete all persistent information associated with the |registration_id|.
   data_manager_->DeleteRegistration(
       registration_id, base::BindOnce(&RecordRegistrationDeletedError));
 
   // Delete the local state associated with the |registration_id|.
   active_fetches_.erase(registration_id);
 }
 
 }  // namespace content