Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5255)

Unified Diff: chrome/browser/profiles/profile_impl.cc

Issue 12077030: Allow signin to continue even if sync is disabled by policy. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed mac tests. Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/profiles/profile_impl.cc
diff --git a/chrome/browser/profiles/profile_impl.cc b/chrome/browser/profiles/profile_impl.cc
index 2760b1d434368509c2e7047541ae9500989f560f..7f8d2272a1c7438d99ad3ce7e2ca2a8f57d75bb4 100644
--- a/chrome/browser/profiles/profile_impl.cc
+++ b/chrome/browser/profiles/profile_impl.cc
@@ -881,6 +881,13 @@ net::URLRequestContextGetter* ProfileImpl::GetRequestContextForStoragePartition(
}
net::SSLConfigService* ProfileImpl::GetSSLConfigService() {
+ // If ssl_config_service_manager_ is null, this typically means that some
+ // ProfileKeyedService is trying to create a RequestContext at startup, but
+ // SSLConfigServiceManager is not initialized until DoFinalInit() which is
+ // invoked after all ProfileKeyedServices have been initialized (see
+ // http://crbug.com/171406).
+ DCHECK(ssl_config_service_manager_.get()) <<
+ "SSLConfigServiceManager is not initialized yet";
return ssl_config_service_manager_->Get();
}

Powered by Google App Engine
This is Rietveld 408576698