[Cleanup] Used scoped pointers in KeyedServiceFactory's SetTestingFactory functions.

chrome/browser/download/download_ui_controller_unittest.cc

 // Copyright 2013 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 "base/bind.h"
 #include "base/callback.h"
 #include "base/files/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 117 matching lines @@
     HistoryService::DownloadQueryCallback download_query_callback_;
 
    private:
     void QueryDownloads(
         const HistoryService::DownloadQueryCallback& callback) override {
       download_query_callback_ = callback;
     }
   };
 
   // Constructs and returns a TestDownloadService.
-  static KeyedService* TestingDownloadServiceFactory(
+  static scoped_ptr<KeyedService> TestingDownloadServiceFactory(
       content::BrowserContext* browser_context);
 
   scoped_ptr<MockDownloadManager> manager_;
   content::DownloadManager::Observer* download_history_manager_observer_;
   content::DownloadManager::Observer* manager_observer_;
   content::DownloadItem* notified_item_;
   base::WeakPtrFactory<content::DownloadItem*> notified_item_receiver_factory_;
 
   HistoryAdapter* history_adapter_;
 };
 
 // static
-KeyedService* DownloadUIControllerTest::TestingDownloadServiceFactory(
+scoped_ptr<KeyedService>
+DownloadUIControllerTest::TestingDownloadServiceFactory(
     content::BrowserContext* browser_context) {
-  return new TestDownloadService(Profile::FromBrowserContext(browser_context));
+  return make_scoped_ptr(
+      new TestDownloadService(Profile::FromBrowserContext(browser_context)));
 }
 
 DownloadUIControllerTest::DownloadUIControllerTest()
     : download_history_manager_observer_(NULL),
       manager_observer_(NULL),
       notified_item_(NULL),
       notified_item_receiver_factory_(&notified_item_) {
 }
 
 void DownloadUIControllerTest::SetUp() {
@@ skipping 174 matching lines @@
   // DownloadUIController. It should ignore the download since it's marked as
   // being restored from history.
   ASSERT_TRUE(manager_observer());
   manager_observer()->OnDownloadCreated(manager(), item.get());
 
   // Finally, the expectation we've been waiting for:
   EXPECT_FALSE(notified_item());
 }
 
 } // namespace