Add the JobComplete callback and error/interrupt information

content/browser/background_fetch/background_fetch_context.h

Index: content/browser/background_fetch/background_fetch_context.h
diff --git a/content/browser/background_fetch/background_fetch_context.h b/content/browser/background_fetch/background_fetch_context.h
index 0ef5d6d7f3b6e4cb3f12992c66e4cdcc4da01254..dbf07c9fc00f1f6248a8485002e30e74fed1de47 100644
--- a/content/browser/background_fetch/background_fetch_context.h
+++ b/content/browser/background_fetch/background_fetch_context.h
@@ -58,6 +58,11 @@ class CONTENT_EXPORT BackgroundFetchContext
   void CreateRequest(const BackgroundFetchJobInfo& job_info,
                      std::vector<BackgroundFetchRequestInfo>& request_infos);
 
+  // Callback for the JobController when the job is complete.
+  // TODO(harkness): Pass back enough information that the context can construct
+  // the Mojo response.

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

micro nit: I'd drop this TODO, it's not really settleable by itself.

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

Done.

+  void JobComplete(const std::string& job_guid);

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

nit: We tend to name callbacks as DidCompleteJob or OnJobCompleted.

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

Done.

+
   void ShutdownOnIO();
 
   // |this| is owned by the BrowserContext via the StoragePartition.