Implements an UpdateClient::SendUninstallPing API for the

components/update_client/ping_manager.cc

 // Copyright 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/update_client/ping_manager.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <vector>
@@ skipping 63 matching lines @@
       StringAppendF(&event, " download_time_ms=\"%s\"",
                     base::Uint64ToString(metrics.download_time_ms).c_str());
     }
     StringAppendF(&event, "/>");
 
     download_events += event;
   }
   return download_events;
 }
 
-// Returns a string representing one ping event xml element for an update item.
+// Returns a string representing one ping event for the update of an item.
+// The event type for this ping event is 3.
 std::string BuildUpdateCompleteEventElement(const CrxUpdateItem* item) {
   DCHECK(item->state == CrxUpdateItem::State::kNoUpdate ||
          item->state == CrxUpdateItem::State::kUpdated);
 
   using base::StringAppendF;
 
   std::string ping_event("<event eventtype=\"3\"");
   const int event_result = item->state == CrxUpdateItem::State::kUpdated;
   StringAppendF(&ping_event, " eventresult=\"%d\"", event_result);
   if (item->error_category)
@@ skipping 15 matching lines @@
                   item->diff_extra_code1);
   }
   if (!item->previous_fp.empty())
     StringAppendF(&ping_event, " previousfp=\"%s\"", item->previous_fp.c_str());
   if (!item->next_fp.empty())
     StringAppendF(&ping_event, " nextfp=\"%s\"", item->next_fp.c_str());
   StringAppendF(&ping_event, "/>");
   return ping_event;
 }
 
+// Returns a string representing one ping event for the uninstall of an item.
+// The event type for this ping event is 4.
+std::string BuildUninstalledEventElement(const CrxUpdateItem* item) {
+  DCHECK(item->state == CrxUpdateItem::State::kUninstalled);
+
+  using base::StringAppendF;
+
+  std::string ping_event("<event eventtype=\"4\" eventresult=\"1\"");
+  if (item->error_code)
+    StringAppendF(&ping_event, " errorcode=\"%d\"", item->error_code);
+  StringAppendF(&ping_event, "/>");
+  return ping_event;
+}
+
 // Builds a ping message for the specified update item.
 std::string BuildPing(const Configurator& config, const CrxUpdateItem* item) {
   const char app_element_format[] =
       "<app appid=\"%s\" version=\"%s\" nextversion=\"%s\">"

[waffles, 2016/01/12 02:26:38]

Just a note that we prefer <event nextversion> to <app nextversion>, but let's
not fix it this changelist.

[Sorin Jianu, 2016/01/12 03:24:06]

Acknowledged.

       "%s"
       "%s"
       "</app>";
+
+  std::string ping_event;
+  switch (item->state) {
+    case CrxUpdateItem::State::kNoUpdate:  // Fall through.
+    case CrxUpdateItem::State::kUpdated:
+      ping_event = BuildUpdateCompleteEventElement(item);
+      break;
+    case CrxUpdateItem::State::kUninstalled:
+      ping_event = BuildUninstalledEventElement(item);
+      break;
+    default:
+      NOTREACHED();
+      break;
+  }
+
   const std::string app_element(base::StringPrintf(
       app_element_format,
       item->id.c_str(),                                    // "appid"
       item->previous_version.GetString().c_str(),          // "version"
       item->next_version.GetString().c_str(),              // "nextversion"
-      BuildUpdateCompleteEventElement(item).c_str(),       // update event
+      ping_event.c_str(),                                  // ping event
       BuildDownloadCompleteEventElements(item).c_str()));  // download events
 
   return BuildProtocolRequest(config.GetBrowserVersion().GetString(),
                               config.GetChannel(), config.GetLang(),
                               config.GetOSLongName(), app_element, "");
 }
 
 // Sends a fire and forget ping. The instances of this class have no
 // ownership and they self-delete upon completion. One instance of this class
 // can send only one ping.
@@ skipping 45 matching lines @@
 }  // namespace
 
 PingManager::PingManager(const scoped_refptr<Configurator>& config)
     : config_(config) {}
 
 PingManager::~PingManager() {
 }
 
 // Sends a fire and forget ping when the updates are complete. The ping
 // sender object self-deletes after sending the ping has completed asynchrously.
-void PingManager::OnUpdateComplete(const CrxUpdateItem* item) {
+void PingManager::SendPing(const CrxUpdateItem* item) {
   PingSender* ping_sender(new PingSender(config_));
   if (!ping_sender->SendPing(item))
     delete ping_sender;
 }
 
 }  // namespace update_client