Convert ProtocolHandlerRegistry to be a ProfileKeyedService.

chrome/test/base/testing_profile.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 "chrome/test/base/testing_profile.h"
 
 #include "build/build_config.h"
 
 #include "base/base_paths.h"
 #include "base/command_line.h"
@@ skipping 646 matching lines @@
 
 bool TestingProfile::IsSameProfile(Profile *p) {
   return this == p;
 }
 
 base::Time TestingProfile::GetStartTime() const {
   return start_time_;
 }
 
 ProtocolHandlerRegistry* TestingProfile::GetProtocolHandlerRegistry() {
-  return protocol_handler_registry_.get();
+  return protocol_handler_registry_;

[Elliot Glaysher, 2012/06/11 19:43:16]

Edit: Now that I look at this, you may just be able to return NULL here;
CreateProtocolHandlerRegistry() looks like dead code.

Previous: There's testing infrastructure for conditionally building a service
during only certain tests. See uses of
ProfileKeyedServiceFactory::SetTestingFactory() for more details. (Or look at
how it's used in autofill_manager_unittest.cc)

[Steve McKay, 2012/06/11 22:09:38]

Done.

 }
 
 FilePath TestingProfile::last_selected_directory() {
   return last_selected_directory_;
 }
 
 void TestingProfile::set_last_selected_directory(const FilePath& path) {
   last_selected_directory_ = path;
 }
 
@@ skipping 35 matching lines @@
 }
 
 bool TestingProfile::WasCreatedByVersionOrLater(const std::string& version) {
   return true;
 }
 
 base::Callback<ChromeURLDataManagerBackend*(void)>
     TestingProfile::GetChromeURLDataManagerBackendGetter() const {
   return base::Callback<ChromeURLDataManagerBackend*(void)>();
 }