Network traffic annotation added to one_google_bar.

chrome/browser/search/one_google_bar/one_google_bar_fetcher_impl.cc

Index: chrome/browser/search/one_google_bar/one_google_bar_fetcher_impl.cc
diff --git a/chrome/browser/search/one_google_bar/one_google_bar_fetcher_impl.cc b/chrome/browser/search/one_google_bar/one_google_bar_fetcher_impl.cc
index 41cdec8f7afde09915d29b5f6028ec1fdb87c0cb..28f1b09925dfedaa4f2961c5d95cd23ca96af0e5 100644
--- a/chrome/browser/search/one_google_bar/one_google_bar_fetcher_impl.cc
+++ b/chrome/browser/search/one_google_bar/one_google_bar_fetcher_impl.cc
@@ -27,6 +27,7 @@
 #include "google_apis/google_api_keys.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_fetcher_delegate.h"
 
@@ -268,7 +269,28 @@ void OneGoogleBarFetcherImpl::AuthenticatedURLFetcher::GotAccessToken(
 
   bool use_oauth = !access_token.empty();
   GURL url = GetApiUrl(use_oauth);
-  url_fetcher_ = net::URLFetcher::Create(0, url, net::URLFetcher::POST, this);
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("...", R"(

[sfiera, 2017/05/22 12:48:17]

one_google_bar_service

[Ramin Halavati, 2017/05/22 13:11:48]

Done.

+        semantics {
+          sender: "..."

[sfiera, 2017/05/22 12:48:17]

One Google Bar Service

[Ramin Halavati, 2017/05/22 13:11:48]

Done.

+          description: "..."

[sfiera, 2017/05/22 12:48:16]

Downloads the "One Google" bar if Google is the configured search provider and
the user is signed in to a Google account.

[sfiera, 2017/05/22 12:51:37]

Ah, with another look, I see we reference an API key in this file, which means
that we do it for signed-out users and this part is incorrect.

[Ramin Halavati, 2017/05/22 13:11:48]

I am confused, could you please rephrase?

+          trigger: "..."

[sfiera, 2017/05/22 12:48:17]

Displaying the new tab page on Desktop

[Ramin Halavati, 2017/05/22 13:11:48]

Done.

+          data: "..."

[sfiera, 2017/05/22 12:48:17]

Credentials of signed-in user

[Ramin Halavati, 2017/05/22 13:11:48]

Done.

+          destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL
+        }
+        policy {
+          cookies_allowed: false
+          setting: "..."

[sfiera, 2017/05/22 12:48:16]

Signing out of Chrome or choosing a non-Google search engine in Chrome settings
under 'Search Engine' disables this feature.

[Ramin Halavati, 2017/05/22 13:11:48]

Done.

+          chrome_policy {
+            [POLICY_NAME] {
+              policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+              [POLICY_NAME]: ... //(value to disable it)
+            }

[sfiera, 2017/05/22 12:48:17]

DefaultSearchProviderEnabled {
  policy_options {mode: MANDATORY}
  DefaultSearchProviderEnabled: false
}

[Ramin Halavati, 2017/05/22 13:11:49]

Done.

+          }
+          policy_exception_justification: "..."
+        })");
+  url_fetcher_ = net::URLFetcher::Create(0, url, net::URLFetcher::POST, this,
+                                         traffic_annotation);
   url_fetcher_->SetRequestContext(request_context_);
 
   url_fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SEND_AUTH_DATA |