Use std::unique_ptr for access to BackgroundFetchJobInfo

content/browser/background_fetch/background_fetch_job_info.h

Index: content/browser/background_fetch/background_fetch_job_info.h
diff --git a/content/browser/background_fetch/background_fetch_job_info.h b/content/browser/background_fetch/background_fetch_job_info.h
index 06487e1b3f666bc7d63e4ef46a869ada3d7118b6..69b929ce890db744eab8f04aa7ebc0f088e696e8 100644
--- a/content/browser/background_fetch/background_fetch_job_info.h
+++ b/content/browser/background_fetch/background_fetch_job_info.h
@@ -8,6 +8,7 @@
 #include <string>
 #include <vector>
 
+#include "base/macros.h"
 #include "content/common/content_export.h"
 #include "content/common/service_worker/service_worker_types.h"
 #include "url/origin.h"
@@ -21,10 +22,7 @@ class CONTENT_EXPORT BackgroundFetchJobInfo {
   BackgroundFetchJobInfo(const std::string& tag,
                          const url::Origin& origin,
                          int64_t service_worker_registration_id);
-  // TODO(harkness): Remove copy constructor once the final (non-map-based)
-  // state management is in place and make the class DISALLOW_COPY_AND_ASSIGN.
   BackgroundFetchJobInfo();

[Peter Beverloo, 2017/03/20 18:14:05]

We should be able to remove the BackgroundFetchJobInfo() constructor now.

[harkness, 2017/03/20 18:40:26]

Done.

-  BackgroundFetchJobInfo(const BackgroundFetchJobInfo& other);
   ~BackgroundFetchJobInfo();
 
   const std::string& guid() const { return guid_; }
@@ -49,6 +47,8 @@ class CONTENT_EXPORT BackgroundFetchJobInfo {
   // size, current download size, total number of files, number of complete
   // files, (possibly) data to show the notification, (possibly) list of in
   // progress FetchRequests.
+
+  DISALLOW_COPY_AND_ASSIGN(BackgroundFetchJobInfo);
 };
 
 }  // namespace content