[GCM] Adding the OnSendAcknowledgement event

components/gcm_driver/default_gcm_app_handler.cc

 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/gcm_driver/default_gcm_app_handler.h"
 
 #include "base/logging.h"
 #include "net/base/ip_endpoint.h"
 
 namespace gcm {
@@ skipping 19 matching lines @@
              << app_id;
 }
 
 void DefaultGCMAppHandler::OnSendError(
     const std::string& app_id,
     const GCMClient::SendErrorDetails& send_error_details) {
   LOG(ERROR) << "No app handler is found to route send error message for "
              << app_id;
 }
 
+void DefaultGCMAppHandler::OnSendAcknowledged(const std::string& app_id,
+                                              const std::string& message_id) {
+  LOG(ERROR) << "No app handler is found to route send acknoledgement for "

[Nicolas Zea, 2014/08/05 23:39:48]

Will this get hit if we send a message after uninstalling the app? Maybe just
DVLOG if it's expected?

[fgorski, 2014/08/06 16:22:27]

Done. That is a valid scenario, and I updated the whole file to use DVLOGs
instead of LOG(ERROR)

+             << app_id;
+}
+
 void DefaultGCMAppHandler::OnConnected(const net::IPEndPoint& ip_endpoint) {
   // TODO(semenzato): update CrOS NIC state.
   DVLOG(1) << "GCM connected to " << ip_endpoint.ToString();
 }
 
 void DefaultGCMAppHandler::OnDisconnected() {
   // TODO(semenzato): update CrOS NIC state.
   DVLOG(1) << "GCM disconnected";
 }
 
 }  // namespace gcm