Network traffic annotation added to one_google_bar.

chrome/browser/search/one_google_bar/one_google_bar_fetcher_impl.cc

 // Copyright 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/search/one_google_bar/one_google_bar_fetcher_impl.h"
 
 #include <string>
 #include <utility>
 
 #include "base/callback.h"
 #include "base/json/json_writer.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/field_trial_params.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/search/one_google_bar/one_google_bar_data.h"
 #include "chrome/common/chrome_content_client.h"
 #include "chrome/common/chrome_features.h"
 #include "components/google/core/browser/google_url_tracker.h"
 #include "components/safe_json/safe_json_parser.h"
 #include "components/signin/core/browser/access_token_fetcher.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/variations/net/variations_http_headers.h"
 #include "google_apis/gaia/oauth2_token_service.h"
 #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"
 
 namespace {
 
 const char kApiUrl[] = "https://onegoogle-pa.googleapis.com/v1/getbar";
 
 const char kApiKeyFormat[] = "?key=%s";
 
 const char kApiScope[] = "https://www.googleapis.com/auth/onegoogle.api";
@@ skipping 221 matching lines @@
 }
 
 void OneGoogleBarFetcherImpl::AuthenticatedURLFetcher::GotAccessToken(
     const GoogleServiceAuthError& error,
     const std::string& access_token) {
   // Delete the token fetcher after we leave this method.
   std::unique_ptr<AccessTokenFetcher> deleter(std::move(token_fetcher_));
 
   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("one_google_bar_service", R"(
+        semantics {
+          sender: "One Google Bar Service"
+          description: "Downloads the 'One Google' bar."
+          trigger:
+            "Displaying the new tab page on Desktop, if Google is the "
+            "configured search provider."
+          data: "Credentials if user has signed-in."

[msramek, 2017/05/22 16:51:16]

nit: AFAIU "signed-in" (with the dash) is an adjective. Therefore, this should
be "has signed in" or, as we usually write, "is signed in".

[Ramin Halavati, 2017/05/23 04:54:57]

Done.

+          destination: GOOGLE_OWNED_SERVICE
+        }
+        policy {
+          cookies_allowed: false
+          setting:
+            "Users can control this feature via selecting a non-Google default "
+            "search engine in Chrome settings under 'Search Engine'."
+          chrome_policy {
+            DefaultSearchProviderEnabled {
+              policy_options {mode: MANDATORY}
+              DefaultSearchProviderEnabled: false
+            }
+          }
+        })");
+  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 |
                              net::LOAD_DO_NOT_SEND_COOKIES |
                              net::LOAD_DO_NOT_SAVE_COOKIES);
   url_fetcher_->SetUploadData("application/json", GetRequestBody());
   url_fetcher_->SetExtraRequestHeaders(
       GetExtraRequestHeaders(url, access_token));
 
   url_fetcher_->Start();
@@ skipping 87 matching lines @@
   Respond(base::nullopt);
 }
 
 void OneGoogleBarFetcherImpl::Respond(
     const base::Optional<OneGoogleBarData>& data) {
   for (auto& callback : callbacks_) {
     std::move(callback).Run(data);
   }
   callbacks_.clear();
 }