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

Unified Diff: chrome/service/cloud_print/cloud_print_proxy_backend.cc

Issue 10545170: [Sync] Propagate XMPP auth errors to SyncNotifierObservers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix 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: chrome/service/cloud_print/cloud_print_proxy_backend.cc
diff --git a/chrome/service/cloud_print/cloud_print_proxy_backend.cc b/chrome/service/cloud_print/cloud_print_proxy_backend.cc
index e14e33b3a584abe4ee3c9c49de4389f8a4b49449..e3db09062463fa184cbb35588fb7eb5f1e6ecb2b 100644
--- a/chrome/service/cloud_print/cloud_print_proxy_backend.cc
+++ b/chrome/service/cloud_print/cloud_print_proxy_backend.cc
@@ -88,8 +88,8 @@ class CloudPrintProxyBackend::Core
virtual void OnAuthFailed() OVERRIDE;
// notifier::PushClientObserver implementation.
- virtual void OnNotificationStateChange(
- bool notifications_enabled) OVERRIDE;
+ virtual void OnPushClientStateChange(
+ notifier::PushClientState push_client_state) OVERRIDE;
virtual void OnIncomingNotification(
const notifier::Notification& notification) OVERRIDE;
@@ -500,10 +500,10 @@ void CloudPrintProxyBackend::Core::NotifyUnregisterPrinters(
backend_->frontend_->OnUnregisterPrinters(auth_token, printer_ids);
}
-void CloudPrintProxyBackend::Core::OnNotificationStateChange(
- bool notification_enabled) {
+void CloudPrintProxyBackend::Core::OnPushClientStateChange(
+ notifier::PushClientState push_client_state) {
DCHECK(MessageLoop::current() == backend_->core_thread_.message_loop());
- notifications_enabled_ = notification_enabled;
+ notifications_enabled_ = (push_client_state == notifier::NOTIFICATIONS_ON);
if (notifications_enabled_) {
notifications_enabled_since_ = base::TimeTicks::Now();
VLOG(1) << "Notifications for connector " << proxy_id_

Powered by Google App Engine
This is Rietveld 408576698