Network traffic annotation added to CloudPrintUrlFetcher.

chrome/service/cloud_print/cloud_print_proxy.cc

 // Copyright (c) 2012 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 "chrome/service/cloud_print/cloud_print_proxy.h"
 
 #include <stddef.h>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 210 matching lines @@
 }
 
 void CloudPrintProxy::OnUnregisterPrinters(
     const std::string& auth_token,
     const std::list<std::string>& printer_ids) {
   UMA_HISTOGRAM_COUNTS_10000("CloudPrint.UnregisterPrinters",
                              printer_ids.size());
   ShutdownBackend();
   ConnectorSettings settings;
   settings.InitFrom(service_prefs_);
-  wipeout_.reset(new CloudPrintWipeout(this, settings.server_url()));
+  net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation =
+      net::DefinePartialNetworkTrafficAnnotation("cloud_print_proxy",
+                                                 "cloud_print", R"(
+          semantics {
+            description:
+              "Sends a request to Cloud Print to unregister one or more "
+              "printers."
+            trigger:
+              "User request of unregistering printers or policy change."

[msramek, 2017/06/08 21:23:32]

nit: Just to be a bit more explicit, can we say which policy? It's defined in
the other half of the annotation, so it's not obvious here.

[Ramin Halavati, 2017/06/09 04:55:46]

Done, but note that the report that is generated using partial annotations will
have full annotation for each call (two parts will be merged). Therefore what a
reader sees is the complete annotation including policy name, and I just pointed
towards that to avoid duplication and also need to update both parts if policy
changes.

+            data: "OAuth2 token and list of parinter ids to unregister."
+          })");
+  wipeout_.reset(new CloudPrintWipeout(this, settings.server_url(),
+                                       partial_traffic_annotation));
   wipeout_->UnregisterPrinters(auth_token, printer_ids);
 }
 
 void CloudPrintProxy::OnXmppPingUpdated(int ping_timeout) {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   service_prefs_->SetInt(prefs::kCloudPrintXmppPingTimeout, ping_timeout);
   service_prefs_->WritePrefs();
 }
 
 void CloudPrintProxy::OnUnregisterPrintersComplete() {
   wipeout_.reset();
   // Finish disabling cloud print for this user.
   DisableForUser();
 }
 
 void CloudPrintProxy::ShutdownBackend() {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   if (backend_.get())
     backend_->Shutdown();
   backend_.reset();
 }
 
 }  // namespace cloud_print