Store BackgroundFetchRequestInfo in a refcounted pointer

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 <memory>
 #include <vector>
 
 #include "base/files/file_path.h"
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
 #include "content/browser/background_fetch/background_fetch_constants.h"
 #include "content/common/content_export.h"
 #include "content/common/service_worker/service_worker_types.h"
 #include "content/public/browser/download_interrupt_reasons.h"
 #include "content/public/browser/download_item.h"
 #include "url/gurl.h"
 
 namespace content {
 
 // Simple class to encapsulate the components of a fetch request.
-class CONTENT_EXPORT BackgroundFetchRequestInfo {
+// TODO(peter): This can likely change to have a single owner, and thus become
+// an std::unique_ptr<>, when persistent storage has been implemented.
+class CONTENT_EXPORT BackgroundFetchRequestInfo
+    : public base::RefCountedThreadSafe<BackgroundFetchRequestInfo> {
  public:
-  BackgroundFetchRequestInfo();
   BackgroundFetchRequestInfo(int request_index,
                              const ServiceWorkerFetchRequest& fetch_request);
-  // TODO(harkness): Remove copy constructor once the final (non-map-based)
-  // state management is in place.
-  BackgroundFetchRequestInfo(const BackgroundFetchRequestInfo& request);
-  ~BackgroundFetchRequestInfo();
 
   // Returns the index of this request in the larger Background Fetch fetch.
   // Should only be consumed by the BackgroundFetchDataManager.
   int request_index() const { return request_index_; }
 
   // Returns the Fetch API request object this object encapsulates.
   const ServiceWorkerFetchRequest& fetch_request() const {
     return fetch_request_;
   }
 
@@ skipping 18 matching lines @@
   }
 
   int64_t received_bytes() const { return received_bytes_; }
   void set_received_bytes(int64_t received_bytes) {
     received_bytes_ = received_bytes;
   }
 
   bool IsComplete() const;
 
  private:
+  friend class base::RefCountedThreadSafe<BackgroundFetchRequestInfo>;
+
+  ~BackgroundFetchRequestInfo();
+
   // Index of this request within a Background Fetch registration.
   int request_index_ = kInvalidBackgroundFetchRequestIndex;
 
   // The request information provided by the developer.
   ServiceWorkerFetchRequest fetch_request_;
 
   std::string download_guid_;
 
   // The following members do not need to be persisted, they can be reset after
   // a chrome restart.
   DownloadItem::DownloadState state_ =
       DownloadItem::DownloadState::MAX_DOWNLOAD_STATE;
   DownloadInterruptReason interrupt_reason_ =
       DownloadInterruptReason::DOWNLOAD_INTERRUPT_REASON_NONE;
   base::FilePath file_path_;
   int64_t received_bytes_ = 0;
+
+  DISALLOW_COPY_AND_ASSIGN(BackgroundFetchRequestInfo);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_BACKGROUND_FETCH_REQUEST_INFO_H_