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

chrome/browser/sync/test_profile_sync_service.h

 // 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.
 
 #ifndef CHROME_BROWSER_SYNC_TEST_PROFILE_SYNC_SERVICE_H_
 #define CHROME_BROWSER_SYNC_TEST_PROFILE_SYNC_SERVICE_H_
 
 #include <string>
 
 #include "base/callback.h"
@@ skipping 79 matching lines @@
       Profile* profile, base::Closure callback);
 
   ProfileSyncComponentsFactoryMock* components_factory_mock();
 
   syncer::TestIdFactory* id_factory();
 
   // Raise visibility to ease testing.
   using ProfileSyncService::NotifyObservers;
 
  protected:
-  static KeyedService* TestFactoryFunction(content::BrowserContext* profile);
+  static scoped_ptr<KeyedService> TestFactoryFunction(
+      content::BrowserContext* profile);
 
   // Return NULL handle to use in backend initialization to avoid receiving
   // js messages on UI loop when it's being destroyed, which are not deleted
   // and cause memory leak in test.
   syncer::WeakHandle<syncer::JsEventHandler> GetJsEventHandler() override;
 
   bool NeedBackup() const override;
 
  private:
   syncer::TestIdFactory id_factory_;
 };
 
 #endif  // CHROME_BROWSER_SYNC_TEST_PROFILE_SYNC_SERVICE_H_