Add the JobComplete callback and error/interrupt information

content/browser/background_fetch/background_fetch_job_data.h

Index: content/browser/background_fetch/background_fetch_job_data.h
diff --git a/content/browser/background_fetch/background_fetch_job_data.h b/content/browser/background_fetch/background_fetch_job_data.h
index 7cd89d1a358639f3b4667a2b27a57cc78cf67dc2..13f76c774d7fbf0e922631de085dd65bee562e47 100644
--- a/content/browser/background_fetch/background_fetch_job_data.h
+++ b/content/browser/background_fetch/background_fetch_job_data.h
@@ -11,6 +11,8 @@
 #include "base/macros.h"
 #include "content/browser/background_fetch/background_fetch_request_info.h"
 #include "content/common/content_export.h"
+#include "content/public/browser/download_interrupt_reasons.h"
+#include "content/public/browser/download_item.h"
 #include "url/origin.h"
 
 namespace content {
@@ -24,9 +26,12 @@ class CONTENT_EXPORT BackgroundFetchJobData {
   ~BackgroundFetchJobData();
 
   // Called by the JobController to inform the JobData that the given fetch
-  // has completed. The JobData returns a boolean indicating whether there
+  // has been updated. The JobData returns a boolean indicating whether there
   // are more requests to process.
-  bool BackgroundFetchRequestInfoComplete(const std::string& fetch_guid);
+  bool UpdateBackgroundFetchRequestState(

[Peter Beverloo, 2017/03/15 16:53:38]

A method name like "UpdateFooBar" tells me that calling it will somehow update
FooBar. In this case I think saying that "FooBar *did* update" is more
appropriate. What about DidUpdateFooBar?

[harkness, 2017/03/16 11:41:10]

This is telling the RequestInfo that the attribute "state" on the DownloadItem
has changed, and that RequestInfo should update its local copy to match the
change. Technically UpdateBackgroundFetchRequestInfoState would fit the naming,
but I think shortening out the "Info" still makes sense.

+      const std::string& fetch_guid,
+      DownloadItem::DownloadState state,
+      DownloadInterruptReason interrupt_reason);
 
   // Called by the JobController to get a BackgroundFetchRequestInfo to
   // process.