Network traffic annotation added to device_description_fetcher.

chrome/browser/extensions/api/dial/device_description_fetcher.cc

 // Copyright (c) 2017 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/browser/extensions/api/dial/device_description_fetcher.h"
 
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/api/dial/dial_device_data.h"
 #include "chrome/browser/profiles/profile.h"
 #include "content/public/browser/browser_thread.h"
 #include "net/base/load_flags.h"
 #include "net/http/http_response_headers.h"
 #include "net/http/http_status_code.h"
 #include "net/http/http_util.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_request_context_getter.h"
 
 using content::BrowserThread;
 
 constexpr char kApplicationUrlHeaderName[] = "Application-URL";
 constexpr int kMaxRetries = 3;
 // DIAL devices are unlikely to expose uPnP functions other than DIAL, so 256kb
 // should be more than sufficient.
 constexpr int kMaxDescriptionSizeBytes = 262144;
@@ skipping 16 matching lines @@
 }
 
 DeviceDescriptionFetcher::~DeviceDescriptionFetcher() {
   DCHECK(thread_checker_.CalledOnValidThread());
 }
 
 void DeviceDescriptionFetcher::Start() {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(!fetcher_);
 
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("...", R"(

[imcheng, 2017/03/13 19:59:42]

dial_get_device_description

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+        semantics {
+          sender: "..."

[imcheng, 2017/03/13 19:59:42]

dial

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+          description: "..."

[imcheng, 2017/03/13 19:59:42]

Chromium sends a request to a device (such as a smart TV) discovered via the
DIAL (Disovery and Launch) protocol to obtain its device description. Chromium
then uses the device description to determine the capabilities of the device to
be used as a target for casting media content.

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+          trigger: "..."

[imcheng, 2017/03/13 19:59:42]

A new or updated device has been discovered via DIAL in the local network.

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+          data: "..."

[imcheng, 2017/03/13 19:59:42]

A HTTP GET request.

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+          destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER

[imcheng, 2017/03/13 19:59:42]

I suppose this should be WEBSITE as the destination is an IP address of a device
on the local network. It returns a XML response to a HTTP GET request.

[Ramin Halavati, 2017/03/13 21:44:30]

I think it's OTHER, as it is not a website that user interacts with.

+        }
+        policy {
+          cookies_allowed: false/true

[imcheng, 2017/03/13 19:59:42]

false

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+          cookies_store: "..."
+          setting: "..."

[imcheng, 2017/03/13 19:59:42]

This (and the Media Router) can be disabled via the EnableMediaRouter policy.

[Ramin Halavati, 2017/03/13 21:44:30]

So it cannot be disabled by settings?

[imcheng, 2017/03/14 22:48:19]

You can disable it via the media-router flag but not through settings.

[Ramin Halavati, 2017/03/15 06:37:37]

Thank you.

+          chrome_policy {
+            [POLICY_NAME] {

[imcheng, 2017/03/13 19:59:42]

EnableMediaRouter

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+              policy_options {mode: MANDATORY/RECOMMENDED/UNSET}

[imcheng, 2017/03/13 19:59:42]

MANDATORY

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+              [POLICY_NAME]: ... //(value to disable it)

[imcheng, 2017/03/13 19:59:42]

false

[Ramin Halavati, 2017/03/13 21:44:30]

Done.

+            }
+          }
+          policy_exception_justification: "..."
+        })");
   // DIAL returns device descriptions via GET request.
   fetcher_ =
       net::URLFetcher::Create(kURLFetcherIDForTest, device_description_url_,
-                              net::URLFetcher::GET, this);
+                              net::URLFetcher::GET, this, traffic_annotation);
 
   // net::LOAD_BYPASS_PROXY: Proxies almost certainly hurt more cases than they
   //     help.
   // net::LOAD_DISABLE_CACHE: The request should not touch the cache.
   // net::LOAD_DO_NOT_{SAVE,SEND}_COOKIES: The request should not touch cookies.
   // net::LOAD_DO_NOT_SEND_AUTH_DATA: The request should not send auth data.
   fetcher_->SetLoadFlags(net::LOAD_BYPASS_PROXY | net::LOAD_DISABLE_CACHE |
                          net::LOAD_DO_NOT_SAVE_COOKIES |
                          net::LOAD_DO_NOT_SEND_COOKIES |
                          net::LOAD_DO_NOT_SEND_AUTH_DATA);
@@ skipping 76 matching lines @@
     int64_t current,
     int64_t total) {}
 
 void DeviceDescriptionFetcher::ReportError(const std::string& message) {
   std::move(error_cb_).Run(message);
 }
 
 }  // namespace dial
 }  // namespace api
 }  // namespace extensions