Network traffic annotation added to url_provision_fetcher.

content/browser/media/url_provision_fetcher.cc

 // Copyright 2015 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 "content/browser/media/url_provision_fetcher.h"
 
 #include "base/memory/ptr_util.h"
 #include "content/public/browser/provision_fetcher_factory.h"
 #include "media/base/bind_to_current_loop.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_fetcher.h"
 
 using net::URLFetcher;
 
 namespace content {
 
 // Implementation of URLProvisionFetcher.
 
 URLProvisionFetcher::URLProvisionFetcher(
     net::URLRequestContextGetter* context_getter)
     : context_getter_(context_getter) {}
 
 URLProvisionFetcher::~URLProvisionFetcher() {}
 
 void URLProvisionFetcher::Retrieve(
     const std::string& default_url,
     const std::string& request_data,
     const media::ProvisionFetcher::ResponseCB& response_cb) {
   response_cb_ = response_cb;
 
   const std::string request_string =
       default_url + "&signedRequest=" + request_data;
   DVLOG(1) << __func__ << ": request:" << request_string;
 
   DCHECK(!request_);
-  request_ = URLFetcher::Create(GURL(request_string), URLFetcher::POST, this);
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("url_prevision_fetcher", R"(
+        semantics {
+          sender: "ProvisionFetcher for Android MediaDrm"
+          description:
+            "Android MediaDrm (a part of encrypted content playback) requires "
+            "provisioning of the device. This is a request to provisioning "
+            "server."
+          trigger: "NotProvisionedException from MediaDrm"

[Ramin Halavati, 2017/03/09 06:28:47]

Please be a little more specific for a not-software-engineer reader.

+          data: "Opaque request generated by Android framework"

[Ramin Halavati, 2017/03/09 06:28:47]

Ditto.

+          destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER

[Ramin Halavati, 2017/03/09 06:28:47]

Isn't it sent to a website that belongs to Google?

+        }
+        policy {
+          cookies_allowed: false/true

[Ramin Halavati, 2017/03/09 06:28:47]

I don't see disabling cookies in the code. If you don't need them, I can create
a CL that disables it and then we set it to false.

+          cookies_store: "..."
+          setting: "..."
+          policy {
+            [POLICY_NAME] {
+              policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+              value: ...
+            }
+          }
+          policy_exception_justification: "..."

[Ramin Halavati, 2017/03/09 06:28:47]

If there is no chrome policy implemented to prevent this, you can skip this
part.

+        })");
+  request_ = URLFetcher::Create(GURL(request_string), URLFetcher::POST, this,
+                                traffic_annotation);
 
   // SetUploadData is mandatory even if we are not uploading anything.
   request_->SetUploadData("", "");
   request_->AddExtraRequestHeader("User-Agent: Widevine CDM v1.0");
   request_->AddExtraRequestHeader("Content-Type: application/json");
 
   DCHECK(context_getter_);
   request_->SetRequestContext(context_getter_);
 
   request_->Start();
@@ skipping 22 matching lines @@
 
 // Implementation of content public method CreateProvisionFetcher().
 
 std::unique_ptr<media::ProvisionFetcher> CreateProvisionFetcher(
     net::URLRequestContextGetter* context_getter) {
   DCHECK(context_getter);
   return base::MakeUnique<URLProvisionFetcher>(context_getter);
 }
 
 }  // namespace content