diff options
author | atwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-21 05:19:46 +0000 |
---|---|---|
committer | atwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-21 05:19:46 +0000 |
commit | e4a739bbe7eceebc94ff9233e4b1614995e3fbcd (patch) | |
tree | 714964744a87e3ec5bc3a8baa8b5d7450f551b79 /chrome/browser/profiles/profile_dependency_manager.cc | |
parent | a97895c126f6e7afef08c3e58fe454d2ec3944c0 (diff) | |
download | chromium_src-e4a739bbe7eceebc94ff9233e4b1614995e3fbcd.zip chromium_src-e4a739bbe7eceebc94ff9233e4b1614995e3fbcd.tar.gz chromium_src-e4a739bbe7eceebc94ff9233e4b1614995e3fbcd.tar.bz2 |
Revert 118589 - Re-commit r117965 and 117130 with a prospective fix for perf regression.
Reverting due to linux sync integration test crashes, and a memory leak.
BUG=108017
TEST=none
TBR=tim@chromium.org,jamesr@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9141027
TBR=atwilson@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9226042
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118601 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/profiles/profile_dependency_manager.cc')
-rw-r--r-- | chrome/browser/profiles/profile_dependency_manager.cc | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/chrome/browser/profiles/profile_dependency_manager.cc b/chrome/browser/profiles/profile_dependency_manager.cc index 6b54a76..71468fd 100644 --- a/chrome/browser/profiles/profile_dependency_manager.cc +++ b/chrome/browser/profiles/profile_dependency_manager.cc @@ -21,7 +21,6 @@ #include "chrome/browser/search_engines/template_url_service_factory.h" #include "chrome/browser/sessions/session_service_factory.h" #include "chrome/browser/sessions/tab_restore_service_factory.h" -#include "chrome/browser/signin/signin_manager_factory.h" #include "chrome/browser/speech/speech_input_extension_manager.h" #include "chrome/browser/sync/profile_sync_service_factory.h" @@ -77,9 +76,10 @@ void ProfileDependencyManager::CreateProfileServices(Profile* profile, for (std::vector<ProfileKeyedServiceFactory*>::reverse_iterator rit = destruction_order_.rbegin(); rit != destruction_order_.rend(); ++rit) { - if (!profile->IsOffTheRecord()) { + if (!profile->IsOffTheRecord() && !profile->AsTestingProfile()) { // We only register preferences on normal profiles because the incognito - // profile shares the pref service with the normal one. + // profile shares the pref service with the normal one and the testing + // profile will often just insert its own PrefService. (*rit)->RegisterUserPrefsOnProfile(profile); } @@ -156,7 +156,6 @@ void ProfileDependencyManager::AssertFactoriesBuilt() { prerender::PrerenderManagerFactory::GetInstance(); ProfileSyncServiceFactory::GetInstance(); SessionServiceFactory::GetInstance(); - SigninManagerFactory::GetInstance(); SpeechInputExtensionManager::InitializeFactory(); TabRestoreServiceFactory::GetInstance(); TemplateURLServiceFactory::GetInstance(); |