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

Unified Diff: sync/tools/sync_listen_notifications.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
« sync/notifier/chrome_invalidation_client.h ('K') | « sync/sync.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/tools/sync_listen_notifications.cc
diff --git a/sync/tools/sync_listen_notifications.cc b/sync/tools/sync_listen_notifications.cc
index 52e2efa63c6d452ddfc1c96f0bf59cddd786c05e..d160285533bd99df390d638fd8e71fcbe0979bae 100644
--- a/sync/tools/sync_listen_notifications.cc
+++ b/sync/tools/sync_listen_notifications.cc
@@ -38,6 +38,16 @@ class NotificationPrinter : public sync_notifier::SyncNotifierObserver {
NotificationPrinter() {}
virtual ~NotificationPrinter() {}
+ virtual void OnNotificationsEnabled() OVERRIDE {
+ LOG(INFO) << "Notifications enabled";
+ }
+
+ virtual void OnNotificationsDisabled(
+ sync_notifier::NotificationsDisabledReason reason) OVERRIDE {
+ LOG(INFO) << "Notifications disabled with reason "
+ << sync_notifier::NotificationsDisabledReasonToString(reason);
+ }
+
virtual void OnIncomingNotification(
const syncable::ModelTypePayloadMap& type_payloads,
sync_notifier::IncomingNotificationSource source) OVERRIDE {
@@ -51,11 +61,6 @@ class NotificationPrinter : public sync_notifier::SyncNotifierObserver {
}
}
- virtual void OnNotificationStateChange(
- bool notifications_enabled) OVERRIDE {
- LOG(INFO) << "Notifications enabled: " << notifications_enabled;
- }
-
private:
DISALLOW_COPY_AND_ASSIGN(NotificationPrinter);
};
« sync/notifier/chrome_invalidation_client.h ('K') | « sync/sync.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698