Remove scoped_refptr::release which is confusing and causes leaks.

content/public/test/test_file_error_injector.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 "content/public/test/test_file_error_injector.h"
 
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
@@ skipping 295 matching lines @@
 void DownloadFileWithErrorsFactory::ClearErrors() {
   injected_errors_.clear();
 }
 
 TestFileErrorInjector::TestFileErrorInjector(
     scoped_refptr<DownloadManager> download_manager)
     : created_factory_(NULL),
       // This code is only used for browser_tests, so a
       // DownloadManager is always a DownloadManagerImpl.
       download_manager_(
-          static_cast<DownloadManagerImpl*>(download_manager.release())) {
+          static_cast<DownloadManagerImpl*>(download_manager.get())) {
   // Record the value of the pointer, for later validation.
   created_factory_ =
       new DownloadFileWithErrorsFactory(
           base::Bind(&TestFileErrorInjector::RecordDownloadFileConstruction,
                      this),
           base::Bind(&TestFileErrorInjector::RecordDownloadFileDestruction,
                      this));
 
   // We will transfer ownership of the factory to the download manager.
   scoped_ptr<DownloadFileFactory> download_file_factory(
@@ skipping 112 matching lines @@
     case FILE_OPERATION_RENAME_ANNOTATE:
       return "RENAME_ANNOTATE";
     default:
       break;
   }
 
   return "Unknown";
 }
 
 }  // namespace content