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("...", 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 |
                              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();
 }