Convert implicit scoped_refptr constructor calls to explicit ones, part 1

webkit/blob/blob_url_request_job_unittest.cc

 // Copyright (c) 2010 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 <stack>
 #include <utility>
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/scoped_temp_dir.h"
@@ skipping 253 matching lines @@
     EXPECT_EQ(request_->response_headers()->response_code(),
               expected_status_code_);
     EXPECT_STREQ(url_request_delegate_->response_data().c_str(),
                  expected_response_.c_str());
     TestFinished();
   }
 
   // Test Cases ---------------------------------------------------------------
 
   void TestGetSimpleDataRequest() {
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     blob_data->AppendData(kTestData1);
     TestSuccessRequest(blob_data, kTestData1);
   }
 
   void TestGetSimpleFileRequest() {
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     blob_data->AppendFile(temp_file1_, 0, -1, base::Time());
     TestSuccessRequest(blob_data, kTestFileData1);
   }
 
   void TestGetNonExistentFileRequest() {
     FilePath non_existent_file =
         temp_file1_.InsertBeforeExtension(FILE_PATH_LITERAL("-na"));
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     blob_data->AppendFile(non_existent_file, 0, -1, base::Time());
     TestErrorRequest(blob_data, 404);
   }
 
   void TestGetChangedFileRequest() {
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     base::Time old_time =
         temp_file_modification_time1_ - base::TimeDelta::FromSeconds(10);
     blob_data->AppendFile(temp_file1_, 0, 3, old_time);
     TestErrorRequest(blob_data, 404);
   }
 
   void TestGetSlicedDataRequest() {
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     blob_data->AppendData(kTestData2, 2, 4);
     std::string result(kTestData2 + 2, 4);
     TestSuccessRequest(blob_data, result);
   }
 
   void TestGetSlicedFileRequest() {
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     blob_data->AppendFile(temp_file1_, 2, 4, temp_file_modification_time1_);
     std::string result(kTestFileData1 + 2, 4);
     TestSuccessRequest(blob_data, result);
   }
 
   scoped_refptr<BlobData> BuildComplicatedData(std::string* expected_result) {
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     blob_data->AppendData(kTestData1, 1, 2);
     blob_data->AppendFile(temp_file1_, 2, 3, temp_file_modification_time1_);
     blob_data->AppendData(kTestData2, 3, 4);
     blob_data->AppendFile(temp_file2_, 4, 5, temp_file_modification_time2_);
     *expected_result = std::string(kTestData1 + 1, 2);
     *expected_result += std::string(kTestFileData1 + 2, 3);
     *expected_result += std::string(kTestData2 + 3, 4);
     *expected_result += std::string(kTestFileData2 + 4, 5);
     return blob_data;
   }
@@ skipping 22 matching lines @@
     net::HttpRequestHeaders extra_headers;
     extra_headers.SetHeader(net::HttpRequestHeaders::kRange, "bytes=-10");
     PushNextTask(NewRunnableMethod(
         this, &BlobURLRequestJobTest::VerifyResponse));
     expected_status_code_ = 206;
     expected_response_ = result.substr(result.length() - 10);
     return TestRequest("GET", extra_headers, blob_data);
   }
 
   void TestExtraHeaders() {
-    scoped_refptr<BlobData> blob_data = new BlobData();
+    scoped_refptr<BlobData> blob_data(new BlobData());
     blob_data->set_content_type(kTestContentType);
     blob_data->set_content_disposition(kTestContentDisposition);
     blob_data->AppendData(kTestData1);
     PushNextTask(NewRunnableMethod(
         this, &BlobURLRequestJobTest::VerifyResponseForTestExtraHeaders));
     TestRequest("GET", net::HttpRequestHeaders(), blob_data);
   }
 
   void VerifyResponseForTestExtraHeaders() {
     EXPECT_TRUE(request_->status().is_success());
@@ skipping 69 matching lines @@
 
 TEST_F(BlobURLRequestJobTest, TestExtraHeaders) {
   RunTestOnIOThread(&BlobURLRequestJobTest::TestExtraHeaders);
 }
 
 }  // namespace webkit_blob
 
 // BlobURLRequestJobTest is expected to always live longer than the
 // runnable methods.  This lets us call NewRunnableMethod on its instances.
 DISABLE_RUNNABLE_METHOD_REFCOUNT(webkit_blob::BlobURLRequestJobTest);