Added DownloadItem::Observer to JobController.

content/browser/background_fetch/background_fetch_request_info.h

Index: content/browser/background_fetch/background_fetch_request_info.h
diff --git a/content/browser/background_fetch/background_fetch_request_info.h b/content/browser/background_fetch/background_fetch_request_info.h
index 434ae070b5023d2759063397a834e6a920c73ad5..15ce167856a9ef95202a304376ce3265c754df08 100644
--- a/content/browser/background_fetch/background_fetch_request_info.h
+++ b/content/browser/background_fetch/background_fetch_request_info.h
@@ -29,11 +29,17 @@ class CONTENT_EXPORT BackgroundFetchRequestInfo {
   bool complete() const { return complete_; }
   void set_complete(bool complete) { complete_ = complete; }
 
+  const std::string& download_guid() const { return download_guid_; }
+  void set_download_guid(const std::string& download_guid) {
+    download_guid_ = download_guid;
+  }

[Peter Beverloo, 2017/03/10 13:40:52]

qq: will we be able to delete one of the GUIDs? This'll get confusing.

[harkness, 2017/03/10 14:21:01]

This is an issue with the DownloadManager. It creates a new guid when we send a
request, and all communication with the DownloadManager is via that guid. I've
discussed with David finding a way that we can request a guid rather than being
handed one.

+
  private:
   std::string guid_;
   GURL url_;
   std::string tag_;
   bool complete_ = false;
+  std::string download_guid_;
 };
 
 using BackgroundFetchRequestInfos = std::vector<BackgroundFetchRequestInfo>;