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

Unified Diff: sync/notifier/invalidation_notifier.cc

Issue 10545170: [Sync] Propagate XMPP auth errors to SyncNotifierObservers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix deps, win compile error Created 8 years, 6 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: sync/notifier/invalidation_notifier.cc
diff --git a/sync/notifier/invalidation_notifier.cc b/sync/notifier/invalidation_notifier.cc
index 485e9694d4953d9804c1463e73e7febc3bbd4c63..9925041450dada1f13dcebae875e64740bd66fc2 100644
--- a/sync/notifier/invalidation_notifier.cc
+++ b/sync/notifier/invalidation_notifier.cc
@@ -102,14 +102,23 @@ void InvalidationNotifier::SendNotification(
void InvalidationNotifier::OnInvalidate(
const syncable::ModelTypePayloadMap& type_payloads) {
DCHECK(CalledOnValidThread());
+ FOR_EACH_OBSERVER(
+ SyncNotifierObserver, observers_,
+ OnIncomingNotification(type_payloads,
+ sync_notifier::REMOTE_NOTIFICATION));
+}
+
+void InvalidationNotifier::OnNotificationsEnabled() {
+ DCHECK(CalledOnValidThread());
FOR_EACH_OBSERVER(SyncNotifierObserver, observers_,
- OnIncomingNotification(type_payloads,
- sync_notifier::REMOTE_NOTIFICATION));
+ OnNotificationsEnabled());
}
-void InvalidationNotifier::OnSessionStatusChanged(bool has_session) {
+void InvalidationNotifier::OnNotificationsDisabled(
+ NotificationsDisabledReason reason) {
+ DCHECK(CalledOnValidThread());
FOR_EACH_OBSERVER(SyncNotifierObserver, observers_,
- OnNotificationStateChange(has_session));
+ OnNotificationsDisabled(reason));
}
} // namespace sync_notifier

Powered by Google App Engine
This is Rietveld 408576698