Removed Profile::GetHostContentSettingsMap

chrome/browser/push_messaging/push_messaging_service_impl.cc

Index: chrome/browser/push_messaging/push_messaging_service_impl.cc
diff --git a/chrome/browser/push_messaging/push_messaging_service_impl.cc b/chrome/browser/push_messaging/push_messaging_service_impl.cc
index f5b7bcd2afce888f7c29414fc18cb29f30cf25a2..515d813a2ef6c14e807994b7d033a41e0111139d 100644
--- a/chrome/browser/push_messaging/push_messaging_service_impl.cc
+++ b/chrome/browser/push_messaging/push_messaging_service_impl.cc
@@ -745,7 +745,7 @@ void PushMessagingServiceImpl::SetContentSettingChangedCallbackForTesting(
 
 void PushMessagingServiceImpl::Shutdown() {
   GetGCMDriver()->RemoveAppHandler(kPushMessagingAppIdentifierPrefix);
-  profile_->GetHostContentSettingsMap()->RemoveObserver(this);
+  HostContentSettingsMapFactory::GetForProfile(profile_)->RemoveObserver(this);

[Peter Beverloo, 2015/09/09 21:03:51]

Shouldn't the PushMessagingServiceFactory declare a dependency on the
HostContentSettingsMapFactory after this change? (Which also goes for many of
the other uses...)

[Bernhard Bauer, 2015/09/10 09:30:44]

Yes, that would be good.

[PEConn, 2015/09/11 12:57:26]

PushMessagingServiceFactory already declares a dependency (it was in the earlier
changelist). However I went through and found some other Factories that needed
dependencies declared.

 #if defined(ENABLE_BACKGROUND)
   // TODO(mvanouwerkerk): Ensure Push API unregisters correctly from Background
   // Mode - crbug.com/527036.