Index: components/gcm_driver/gcm_client_impl.cc |
diff --git a/components/gcm_driver/gcm_client_impl.cc b/components/gcm_driver/gcm_client_impl.cc |
index 1af8dc5023b8e73264ccee853c79d53fa4f31006..b5240f95ea4ca1c85f89b218d350948a7c22a0b0 100644 |
--- a/components/gcm_driver/gcm_client_impl.cc |
+++ b/components/gcm_driver/gcm_client_impl.cc |
@@ -857,15 +857,15 @@ void GCMClientImpl::OnMessageSentToMCS(int64 user_serial_number, |
// All other errors will be raised immediately, through asynchronous callback. |
// It is expected that TTL_EXCEEDED will be issued for a message that was |
// previously issued |OnSendFinished| with status SUCCESS. |
- // For now, we do not report that the message has been sent and acked |
- // successfully. |
// TODO(jianli): Consider adding UMA for this status. |
if (status == MCSClient::TTL_EXCEEDED) { |
SendErrorDetails send_error_details; |
send_error_details.message_id = message_id; |
send_error_details.result = GCMClient::TTL_EXCEEDED; |
delegate_->OnMessageSendError(app_id, send_error_details); |
- } else if (status != MCSClient::SENT) { |
+ } else if (status == MCSClient::SENT) { |
+ delegate_->OnSendAcknowledged(app_id, message_id); |
+ } else { |
delegate_->OnSendFinished(app_id, message_id, ToGCMClientResult(status)); |
} |
} |