Network traffic annotation added to translate_url_fetcher.

components/translate/core/browser/translate_url_fetcher.cc

 // Copyright 2014 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 "components/translate/core/browser/translate_url_fetcher.h"
 
 #include "components/data_use_measurement/core/data_use_user_data.h"
 #include "components/translate/core/browser/translate_download_manager.h"
 #include "net/base/load_flags.h"
 #include "net/http/http_status_code.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_request_status.h"
 
 namespace translate {
 
 namespace {
 
 // Retry parameter for fetching.
 const int kMaxRetry = 16;
 
@@ skipping 18 matching lines @@
   }
 
   if (retry_count_ >= kMaxRetry)
     return false;
   retry_count_++;
 
   state_ = REQUESTING;
   url_ = url;
   callback_ = callback;
 
-  fetcher_ = net::URLFetcher::Create(id_, url_, net::URLFetcher::GET, this);
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("translate_url_fetcher", R"(
+        semantics {
+          sender: "Translate"
+          description:
+            "Chromium can provide translations for the web sites visited by "

[Takashi Toyoshima, 2017/04/18 07:27:36]

Sorry, one more nits.
According to the style guideline, we should not use "Chromium" in the code base,
but use "Chrome" instead, even for the open sourced builds.

https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md...

[Ramin Halavati, 2017/04/18 07:58:00]

You are right. We had missed this rule. I will change this CL and will create
another CL in future to revert all Chromiums to Chrome.

+            "the user. If this feature is enabled, Chromium sends network "

[Takashi Toyoshima, 2017/04/18 07:27:36]

ditto

[Ramin Halavati, 2017/04/18 07:58:00]

Done.

+            "requests to download the list of supported languages, a library "
+            "to perform translations, and a predictive model to know when to "
+            "offer translation."
+          trigger:
+            "When Chromium starts, it downloads the list of supported "

[Takashi Toyoshima, 2017/04/18 07:27:36]

ditto

[Ramin Halavati, 2017/04/18 07:58:00]

Done.

+            "langagues for translation, and a predictive model to detect the "
+            "language of web pages. The model is cached and the link to the "
+            "latest model is provided by field study. If latest model is "
+            "already cached, it is not fetched. The first time the model "
+            "decides to offer translation of a web site, it triggers a popup "
+            "to ask if user wants a translation and if user approves, "
+            "translation library is downloaded. The library is cached for a "
+            "day and is not fetched if it is available and fresh."
+          data:
+            "Current locale is sent to fetch the list of supperted lanaguges. "
+            "Translation library that is obtained via this interface would "
+            "perform actual translation, and it will send words and phrases in "
+            "the site to the server to translate it, but this request doesn't "
+            "send any words."
+          destination: GOOGLE_OWNED_SERVICE
+        }
+        policy {
+          cookies_allowed: false
+          setting:
+            "Users can enable/disable this feature by toggling 'Offer to "
+            "translate pages that aren't in a language you read.' in Chromium "

[Takashi Toyoshima, 2017/04/18 07:27:36]

ditto

[Ramin Halavati, 2017/04/18 07:58:00]

Done.

+            "settings under Languages. The list of supported languages is "
+            "downloaded regardless of the settings."
+          chrome_policy {
+            TranslateEnabled {
+              policy_options {mode: MANDATORY}
+              TranslateEnabled: false
+            }
+          }
+          policy_exception_justification:
+            "There is no policy for disabling download of the list of "
+            "supported languages. It is considered not required as the list is "
+            "needed for rendering user interfaces, and Chrome does not send "
+            "privacy/security sensitive data to the server on downloading it."
+        })");
+  fetcher_ = net::URLFetcher::Create(id_, url_, net::URLFetcher::GET, this,
+                                     traffic_annotation);
   data_use_measurement::DataUseUserData::AttachToFetcher(
       fetcher_.get(), data_use_measurement::DataUseUserData::TRANSLATE);
   fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES |
                          net::LOAD_DO_NOT_SAVE_COOKIES);
   fetcher_->SetRequestContext(
       TranslateDownloadManager::GetInstance()->request_context());
   // Set retry parameter for HTTP status code 5xx. This doesn't work against
   // 106 (net::ERR_INTERNET_DISCONNECTED) and so on.
   // TranslateLanguageList handles network status, and implements retry.
   fetcher_->SetMaxRetriesOn5xx(max_retry_on_5xx_);
@@ skipping 16 matching lines @@
   } else {
     state_ = FAILED;
   }
 
   // Transfer URLFetcher's ownership before invoking a callback.
   std::unique_ptr<const net::URLFetcher> delete_ptr(fetcher_.release());
   callback_.Run(id_, state_ == COMPLETED, data);
 }
 
 }  // namespace translate