Network traffic annotation added to chromium_url_request.

remoting/base/telemetry_log_writer.cc

 // Copyright 2016 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 "remoting/base/telemetry_log_writer.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/logging.h"
 #include "net/http/http_status_code.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 
 namespace remoting {
 
 const int kMaxSendAttempts = 5;
 
 TelemetryLogWriter::TelemetryLogWriter(
     const std::string& telemetry_base_url,
     std::unique_ptr<UrlRequestFactory> request_factory)
     : telemetry_base_url_(telemetry_base_url),
       request_factory_(std::move(request_factory)) {}
@@ skipping 39 matching lines @@
   if (!serializer.Serialize(log_dictionary)) {
     LOG(ERROR) << "Failed to serialize log to JSON.";
     return;
   }
   PostJsonToServer(json);
 }
 
 void TelemetryLogWriter::PostJsonToServer(const std::string& json) {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(!request_);
-  request_ = request_factory_->CreateUrlRequest(UrlRequest::Type::POST,
-                                                telemetry_base_url_);
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("CRD_telemetry_log", R"(
+        semantics {
+          sender: "Chrome Remote Desktop"
+          description: "Telemetry logs for Chrome Remote Desktop."
+          trigger: "Chrome Remote Desktop is being used."
+          data: "Anonymous usage statistics."
+          destination: GOOGLE_OWNED_SERVICE
+        }
+        policy {
+          cookies_allowed: false/true
+          cookies_store: "..."
+          setting:
+            "This feature cannot be disabled by settings. You can block Chrome "
+            "Remote Desktop as specified here: "
+            "https://support.google.com/chrome/a/answer/2799701?hl=en"
+          policy_exception_justification:
+            "The product is shipped separately from Chromium, except on Chrome "
+            "OS. In future the same code will be used for the host and we "
+            "could potentially add one."

[Sergey Ulanov, 2017/04/04 00:16:48]

Not sure the second sentence is necessary. 
If you do keep then maybe replace "add one" with "add a policy to control it"

[Ramin Halavati, 2017/04/04 07:37:18]

Done.

+        })");
+  request_ = request_factory_->CreateUrlRequest(
+      UrlRequest::Type::POST, telemetry_base_url_, traffic_annotation);
   if (!auth_token_.empty()) {
     request_->AddHeader("Authorization:Bearer " + auth_token_);
   }
 
   VLOG(1) << "Posting log to telemetry server: " << json;
 
   request_->SetPostData("application/json", json);
   request_->Start(
       base::Bind(&TelemetryLogWriter::OnSendLogResult, base::Unretained(this)));
 }
@@ skipping 25 matching lines @@
   request_.reset();  // This may also destroy the result.
   if (should_call_auth_closure) {
     VLOG(1) << "Request is unauthorized. Trying to call the auth closure...";
     auth_closure_.Run();
   } else {
     SendPendingEntries();
   }
 }
 
 }  // namespace remoting