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 "
+            "the user. If this feature is enabled, Chromium sends network "
+            "requests on startup 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 translates a web site, it triggers a request to "

[Roger McFarlane (Chromium), 2017/04/11 14:04:19]

I'm not sure what level of detail/precision is required... but, here's a
clarification (targeted to the CL author, not the end user; so, some
word-smith-fu required):

Chrome downloads the predictive model on startup. It downloads the language list
and library the first time Chrome considers offering a translation.

[Takashi Toyoshima, 2017/04/12 05:49:18]

Do you think the predictive model downloading can be stopped by the translation
settings and enterprise policy?

I remind that the language detection is exposed to users via Chrome Extension
API. So it probably continues to work even if the translation is disabled?

https://developer.chrome.com/extensions/tabs#method-detectLanguage

[Ramin Halavati, 2017/04/12 05:54:43]

Done.

[Ramin Halavati, 2017/04/12 05:54:44]

Acknowledged.

[Roger McFarlane (Chromium), 2017/04/12 14:04:12]

That's a good idea. But currently no, the download happens if the user's doesn't
have the latest model already cached locally.

Filed http://crbug.com/710851

+            "fetch the supporting language list in the first translation, and "
+            "will trigger another request to fetch the library once a day to "
+            "make sure it is up to date."
+          data:
+            "Language id of the library that is required. Translation library "

[Takashi Toyoshima, 2017/04/11 10:00:06]

Precisely said, Chrome send the current locale on fetching the list of supported
languages. This is for obtaining human readable language names in users'
language (but these strings are not used today, AFAIK).

So, this is not for the library.

[Ramin Halavati, 2017/04/12 05:54:43]

Done.

+            "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 "
+            "settings under Languages."
+          chrome_policy {
+            TranslateEnabled {
+              policy_options {mode: MANDATORY}
+              TranslateEnabled: false
+            }
+          }
+        })");
+  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