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:
+            "Chrome can provide translations for the web sites visited by the "
+            "user. If this feature is enabled, Chrome sends network 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 Chrome starts, it downloads the list of supported langagues "
+            "for translation, and a predictive model to detect the language of "

[Roger McFarlane (Chromium), 2017/04/18 20:48:19]

The predictive model isn't for language detection; it predicts the utility of
prompting the user with the translation (what is the probability the user
actually wants/needs the translation).

The language detection model is statically linked to Chrome (i.e., not
downloaded).

[Ramin Halavati, 2017/04/19 06:07:05]

Done.

[Roger McFarlane (Chromium), 2017/05/03 14:34:09]

I'm not sure why you removed so much text. There seems to be some confusion.

There are two models:

 - language detection model: statically linked to chrome

 - predictive model to infer whether a translation from
   the detected language to some other language would be
   useful/accepted by the user. This is downloaded and
   cached when chrome starts.

The previous version of the annotation seemed to be conflating these two models.
My comment was that the predictive model is not the same thing as the language
detection model (the latter of which is not downloaded).

[Ramin Halavati, 2017/05/03 15:20:40]

Oh sorry, I misunderstood it.

+            "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 Chrome "
+            "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