Chromium Code Reviews| Index: chrome/browser/prefs/pref_notifier_impl.cc |
| diff --git a/chrome/browser/prefs/pref_notifier_impl.cc b/chrome/browser/prefs/pref_notifier_impl.cc |
| index d5ad24c94338f2748ed19ca59291095f808a608b..bc10e53083025567e435173924d02132fc1e4a3b 100644 |
| --- a/chrome/browser/prefs/pref_notifier_impl.cc |
| +++ b/chrome/browser/prefs/pref_notifier_impl.cc |
| @@ -74,13 +74,13 @@ void PrefNotifierImpl::OnPreferenceChanged(const std::string& path) { |
| FireObservers(path); |
| } |
| -void PrefNotifierImpl::OnInitializationCompleted() { |
| +void PrefNotifierImpl::OnInitializationCompleted(bool succeeded) { |
| DCHECK(CalledOnValidThread()); |
| NotificationService::current()->Notify( |
| NotificationType::PREF_INITIALIZATION_COMPLETED, |
| Source<PrefService>(pref_service_), |
| - NotificationService::NoDetails()); |
| + Details<PrefService>(succeeded ? pref_service_ : NULL)); |
|
Mattias Nissler (ping if slow)
2011/05/04 23:33:08
That's a weird contract. Why not have an enum and
altimofeev
2011/05/05 12:59:28
I mistakenly thought that detail's pointer should
|
| } |
| void PrefNotifierImpl::FireObservers(const std::string& path) { |