Remove refcounting from SavePackage.

content/browser/download/save_package_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 <stddef.h>
 #include <stdint.h>
 
+#include <memory>
 #include <string>
 
 #include "base/files/file_path.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/macros.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "content/browser/download/save_package.h"
 #include "content/public/common/url_constants.h"
@@ skipping 80 matching lines @@
   }
 
  protected:
   void SetUp() override {
     RenderViewHostImplTestHarness::SetUp();
 
     // Do the initialization in SetUp so contents() is initialized by
     // RenderViewHostImplTestHarness::SetUp.
     ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
 
-    save_package_success_ = new SavePackage(contents(),
-        temp_dir_.path().AppendASCII("testfile" HTML_EXTENSION),
-        temp_dir_.path().AppendASCII("testfile_files"));
+    save_package_success_ = base::WrapUnique(new SavePackage(
+        contents(), temp_dir_.path().AppendASCII("testfile" HTML_EXTENSION),
+        temp_dir_.path().AppendASCII("testfile_files")));
 
     // We need to construct a path that is *almost* kMaxFilePathLength long
     long_file_name.reserve(kMaxFilePathLength + long_file_name.length());
     while (long_file_name.length() < kMaxFilePathLength)
       long_file_name += long_file_name;
     long_file_name.resize(
         kMaxFilePathLength - 9 - temp_dir_.path().value().length());
 
-    save_package_fail_ = new SavePackage(contents(),
+    save_package_fail_ = base::WrapUnique(new SavePackage(
+        contents(),
         temp_dir_.path().AppendASCII(long_file_name + HTML_EXTENSION),
-        temp_dir_.path().AppendASCII(long_file_name + "_files"));
+        temp_dir_.path().AppendASCII(long_file_name + "_files")));
   }
 
  private:
   // SavePackage for successfully generating file name.
-  scoped_refptr<SavePackage> save_package_success_;
+  std::unique_ptr<SavePackage> save_package_success_;
   // SavePackage for failed generating file name.
-  scoped_refptr<SavePackage> save_package_fail_;
+  std::unique_ptr<SavePackage> save_package_fail_;
 
   base::ScopedTempDir temp_dir_;
 };
 
 static const struct {
   const char* disposition;
   const char* url;
   const base::FilePath::CharType* expected_name;
   bool need_htm_ext;
 } kGeneratedFiles[] = {
@@ skipping 254 matching lines @@
 };
 
 // Crashing on Windows, see http://crbug.com/79365
 #if defined(OS_WIN)
 #define MAYBE_TestSuggestedSaveNames DISABLED_TestSuggestedSaveNames
 #else
 #define MAYBE_TestSuggestedSaveNames TestSuggestedSaveNames
 #endif
 TEST_F(SavePackageTest, MAYBE_TestSuggestedSaveNames) {
   for (size_t i = 0; i < arraysize(kSuggestedSaveNames); ++i) {
-    scoped_refptr<SavePackage> save_package(
+    std::unique_ptr<SavePackage> save_package(
         new SavePackage(contents(), base::FilePath(), base::FilePath()));
     save_package->page_url_ = GURL(kSuggestedSaveNames[i].page_url);
     save_package->title_ = kSuggestedSaveNames[i].page_title;
 
     base::FilePath save_name = save_package->GetSuggestedNameForSaveAs(
         kSuggestedSaveNames[i].ensure_html_extension, std::string());
     EXPECT_EQ(kSuggestedSaveNames[i].expected_name, save_name.value()) <<
         "Test case " << i;
   }
 }
@@ skipping 13 matching lines @@
   GURL mock_url = net::URLRequestMockHTTPJob::GetMockUrl("save_page/a.htm");
   GURL view_source_url =
       GURL(kViewSourceScheme + std::string(":") + mock_url.spec());
   GURL actual_url = net::URLRequestMockHTTPJob::GetMockUrl("save_page/a.htm");
   NavigateAndCommit(view_source_url);
   EXPECT_EQ(actual_url, GetUrlToBeSaved());
   EXPECT_EQ(view_source_url, contents()->GetLastCommittedURL());
 }
 
 }  // namespace content