Added DownloadItem::Observer to JobController.

content/browser/background_fetch/background_fetch_request_info.h

 // 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.
 
 #ifndef CONTENT_BROWSER_BACKGROUND_FETCH_REQUEST_INFO_H_
 #define CONTENT_BROWSER_BACKGROUND_FETCH_REQUEST_INFO_H_
 
 #include <vector>
 
 #include "content/common/content_export.h"
@@ skipping 11 matching lines @@
   BackgroundFetchRequestInfo(const BackgroundFetchRequestInfo& request);
   ~BackgroundFetchRequestInfo();
 
   const std::string& guid() const { return guid_; }
   const GURL& url() const { return url_; }
   const std::string& tag() const { return tag_; }
 
   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>;
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_BACKGROUND_FETCH_REQUEST_INFO_H_