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

chrome/browser/history/web_history_service_unittest.cc

 // Copyright 2014 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 "components/history/core/browser/web_history_service.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/run_loop.h"
 #include "chrome/browser/history/web_history_service_factory.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
@@ skipping 177 matching lines @@
     Request* request) {
   return expected_post_data_[request];
 }
 
 std::string TestingWebHistoryService::GetExpectedAudioHistoryValue() {
   if (expected_audio_history_value_)
     return "true";
   return "false";
 }
 
-static KeyedService* BuildWebHistoryService(content::BrowserContext* context) {
+static scoped_ptr<KeyedService> BuildWebHistoryService(
+    content::BrowserContext* context) {
   Profile* profile = static_cast<Profile*>(context);
-  return new TestingWebHistoryService(
+  return make_scoped_ptr(new TestingWebHistoryService(
       ProfileOAuth2TokenServiceFactory::GetForProfile(profile),
       SigninManagerFactory::GetForProfile(profile),
-      profile->GetRequestContext());
+      profile->GetRequestContext()));
 }
 
 }  // namespace
 
 // A test class used for testing the WebHistoryService class.
 // In order for WebHistoryService to be valid, we must have a valid
 // ProfileSyncService. Using the ProfileSyncServiceMock class allows to
 // assign specific return values as needed to make sure the web history
 // service is available.
 class WebHistoryServiceTest : public testing::Test {
@@ skipping 205 matching lines @@
           "  \"history_recording\": true\n"
           "}"));
   scoped_ptr<base::DictionaryValue> response_value5;
   // ReadResponse deletes the request
   response_value5 = TestingWebHistoryService::ReadResponse(request5.get());
   enabled_value = true;
   EXPECT_FALSE(response_value5->GetBoolean("history_recording_enabled",
                                            &enabled_value));
   EXPECT_TRUE(enabled_value);
 }