Switch SupportsUserData uses to use unique_ptr.

chrome/browser/download/download_path_reservation_tracker.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/download/download_path_reservation_tracker.h"
 
 #include <stddef.h>
 
 #include <map>
 #include <string>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/logging.h"
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/path_service.h"
 #include "base/stl_util.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/third_party/icu/icu_utf.h"
 #include "build/build_config.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/features.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/download_item.h"
@@ skipping 27 matching lines @@
 // that is supposed to overwrite an existing reservation.
 ReservationMap* g_reservation_map = NULL;
 
 // Observes a DownloadItem for changes to its target path and state. Updates or
 // revokes associated download path reservations as necessary. Created, invoked
 // and destroyed on the UI thread.
 class DownloadItemObserver : public DownloadItem::Observer,
                              public base::SupportsUserData::Data {
  public:
   explicit DownloadItemObserver(DownloadItem* download_item);
+  ~DownloadItemObserver() override;
 
  private:
-  ~DownloadItemObserver() override;
-
   // DownloadItem::Observer
   void OnDownloadUpdated(DownloadItem* download) override;
   void OnDownloadDestroyed(DownloadItem* download) override;
 
   DownloadItem* download_item_;
 
   // Last known target path for the download.
   base::FilePath last_target_path_;
 
   static const int kUserDataKey;
@@ skipping 252 matching lines @@
     PathValidationResult result) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   callback.Run(result, *reserved_path);
 }
 
 DownloadItemObserver::DownloadItemObserver(DownloadItem* download_item)
     : download_item_(download_item),
       last_target_path_(download_item->GetTargetFilePath()) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   download_item_->AddObserver(this);
-  download_item_->SetUserData(&kUserDataKey, this);
+  download_item_->SetUserData(&kUserDataKey, base::WrapUnique(this));
 }
 
 DownloadItemObserver::~DownloadItemObserver() {
   download_item_->RemoveObserver(this);
   // DownloadItemObserver is owned by DownloadItem. It should only be getting
   // destroyed because it's being removed from the UserData pool. No need to
   // call DownloadItem::RemoveUserData().
 }
 
 void DownloadItemObserver::OnDownloadUpdated(DownloadItem* download) {
@@ skipping 77 matching lines @@
       base::Bind(&CreateReservation, info, reserved_path),
       base::Bind(&RunGetReservedPathCallback, callback,
                  base::Owned(reserved_path)));
 }
 
 // static
 bool DownloadPathReservationTracker::IsPathInUseForTesting(
     const base::FilePath& path) {
   return IsPathInUse(path);
 }