Profiles: Move NetworkActionPredictor to the ProfileKeyedService framework.

chrome/browser/profiles/profile_dependency_manager.cc

Index: chrome/browser/profiles/profile_dependency_manager.cc
diff --git a/chrome/browser/profiles/profile_dependency_manager.cc b/chrome/browser/profiles/profile_dependency_manager.cc
index 0f4cbacd530602e7d3eca3692d876db9ba0f867f..f3c13c2c7a93fd741a0483c4d5e4b39f6d440167 100644
--- a/chrome/browser/profiles/profile_dependency_manager.cc
+++ b/chrome/browser/profiles/profile_dependency_manager.cc
@@ -8,6 +8,7 @@
 #include <deque>
 #include <iterator>
 
+#include "chrome/browser/autocomplete/network_action_predictor_factory.h"
 #include "chrome/browser/autofill/personal_data_manager_factory.h"
 #include "chrome/browser/background/background_contents_service_factory.h"
 #include "chrome/browser/content_settings/cookie_settings.h"
@@ -42,11 +43,12 @@ void AssertFactoriesBuilt() {
     BackgroundContentsServiceFactory::GetInstance();
     CloudPrintProxyServiceFactory::GetInstance();
     CookieSettings::Factory::GetInstance();
-    SpeechInputExtensionManager::InitializeFactory();
+    NetworkActionPredictorFactory::GetInstance();

[dominich, 2011/12/12 18:25:01]

As this is the only place GetInstance is called, and it's actually used to
initialize the factory, should it be called Initialize instead?

[Elliot Glaysher, 2011/12/12 18:43:47]

This isn't the only place it's used. It's used from the GetForProfile() accessor
and, if anything ever depends on NetworkActionPredictorFactory being created
first, it will be called from a DependsOn() line.

SpeechInputExtensionManager::InitializeFactory() is wrong.

     PersonalDataManagerFactory::GetInstance();
     PluginPrefsFactory::GetInstance();
     prerender::PrerenderManagerFactory::GetInstance();
     SessionServiceFactory::GetInstance();
+    SpeechInputExtensionManager::InitializeFactory();
     TabRestoreServiceFactory::GetInstance();
     TemplateURLServiceFactory::GetInstance();