Network traffic annotation added to CloudPrintUrlFetcher.

chrome/service/cloud_print/cloud_print_proxy.cc

Index: chrome/service/cloud_print/cloud_print_proxy.cc
diff --git a/chrome/service/cloud_print/cloud_print_proxy.cc b/chrome/service/cloud_print/cloud_print_proxy.cc
index 02404069ec95d795704f0946c36523c457cd9b50..342a88cf0d95d4e34caed8a3f066b3437f4a3d78 100644
--- a/chrome/service/cloud_print/cloud_print_proxy.cc
+++ b/chrome/service/cloud_print/cloud_print_proxy.cc
@@ -228,7 +228,19 @@ void CloudPrintProxy::OnUnregisterPrinters(
   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);
 }