Remove stl_util's deletion function use and ScopedVector from chrome/browser/download/.

chrome/browser/download/download_history_unittest.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_history.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <utility>
 #include <vector>
 
 #include "base/guid.h"
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/memory/weak_ptr.h"
 #include "base/rand_util.h"
 #include "base/stl_util.h"
 #include "components/history/content/browser/download_constants_utils.h"
 #include "components/history/core/browser/download_constants.h"
 #include "components/history/core/browser/download_row.h"
 #include "components/history/core/browser/history_service.h"
 #include "content/public/test/mock_download_item.h"
 #include "content/public/test/mock_download_manager.h"
 #include "content/public/test/test_browser_thread.h"
@@ skipping 160 matching lines @@
  public:
   // Generic callback that receives a pointer to a StrictMockDownloadItem.
   typedef base::Callback<void(content::MockDownloadItem*)> DownloadItemCallback;
 
   DownloadHistoryTest()
       : ui_thread_(content::BrowserThread::UI, &loop_),
         manager_(new content::MockDownloadManager()),
         history_(NULL),
         manager_observer_(NULL),
         download_created_index_(0) {}
-  ~DownloadHistoryTest() override { base::STLDeleteElements(&items_); }
+  ~DownloadHistoryTest() override {}
 
  protected:
   void TearDown() override { download_history_.reset(); }
 
   content::MockDownloadManager& manager() { return *manager_.get(); }
   content::MockDownloadItem& item(size_t index) { return *items_[index]; }
   DownloadHistory* download_history() { return download_history_.get(); }
 
   void SetManagerObserver(
       content::DownloadManager::Observer* manager_observer) {
@@ skipping 154 matching lines @@
                 content::DownloadItem::DownloadState state,
                 content::DownloadDangerType danger_type,
                 content::DownloadInterruptReason interrupt_reason,
                 bool opened,
                 const std::string& by_extension_id,
                 const std::string& by_extension_name,
                 history::DownloadRow* info) {
     DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
     size_t index = items_.size();
-    StrictMockDownloadItem* mock_item = new StrictMockDownloadItem();
-    items_.push_back(mock_item);
+    items_.push_back(base::MakeUnique<StrictMockDownloadItem>());
 
     info->current_path = current_path;
     info->target_path = target_path;
     info->url_chain = url_chain;
     info->referrer_url = referrer;
     info->site_url = site_url;
     info->tab_url = tab_url;
     info->tab_referrer_url = tab_referrer_url;
     info->mime_type = mime_type;
     info->original_mime_type = original_mime_type;
@@ skipping 71 matching lines @@
 
  private:
   void CheckDownloadWasRestoredFromHistory(bool expected_value,
                                            content::MockDownloadItem* item) {
     ASSERT_TRUE(download_history_.get());
     EXPECT_EQ(expected_value, download_history_->WasRestoredFromHistory(item));
   }
 
   base::MessageLoopForUI loop_;
   content::TestBrowserThread ui_thread_;
-  std::vector<StrictMockDownloadItem*> items_;
+  std::vector<std::unique_ptr<StrictMockDownloadItem>> items_;
   std::unique_ptr<content::MockDownloadManager> manager_;
   FakeHistoryAdapter* history_;
   std::unique_ptr<DownloadHistory> download_history_;
   content::DownloadManager::Observer* manager_observer_;
   size_t download_created_index_;
   DownloadItemCallback pre_on_create_handler_;
   DownloadItemCallback post_on_create_handler_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadHistoryTest);
 };
@@ skipping 422 matching lines @@
   FinishCreateDownload();
   EXPECT_TRUE(DownloadHistory::IsPersisted(&item(0)));
 
   // ItemAdded should call OnDownloadUpdated, which should detect that the item
   // changed while it was being added and call UpdateDownload immediately.
   info.opened = true;
   ExpectDownloadUpdated(info);
 }
 
 }  // anonymous namespace