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

Unified Diff: components/invalidation/push_client_channel.cc

Issue 348503002: Report InvalidatorState correctly from GCMNetworkChannel (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@GCM.ConnectionState
Patch Set: Rebase Created 6 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: components/invalidation/push_client_channel.cc
diff --git a/components/invalidation/push_client_channel.cc b/components/invalidation/push_client_channel.cc
index 3d12d485a232a8d38f3dd0742fbf2ce1f33caa9c..022993ea3b4d3e954b0cfc4e7858f1fb725fcc22 100644
--- a/components/invalidation/push_client_channel.cc
+++ b/components/invalidation/push_client_channel.cc
@@ -70,12 +70,14 @@ void PushClientChannel::SendMessage(const std::string& message) {
}
void PushClientChannel::OnNotificationsEnabled() {
- NotifyStateChange(INVALIDATIONS_ENABLED);
+ NotifyNetworkStatusChange(true);
+ NotifyChannelStateChange(INVALIDATIONS_ENABLED);
}
void PushClientChannel::OnNotificationsDisabled(
notifier::NotificationsDisabledReason reason) {
- NotifyStateChange(FromNotifierReason(reason));
+ NotifyNetworkStatusChange(false);
+ NotifyChannelStateChange(FromNotifierReason(reason));
}
void PushClientChannel::OnIncomingNotification(

Powered by Google App Engine
This is Rietveld 408576698