Network traffic annotation added to google_apis/gaia.

google_apis/gaia/gaia_auth_fetcher.cc

Index: google_apis/gaia/gaia_auth_fetcher.cc
diff --git a/google_apis/gaia/gaia_auth_fetcher.cc b/google_apis/gaia/gaia_auth_fetcher.cc
index 03bd8a4526d4b3ac930b2a06f26528ce67402351..52640d08a02aa0042c6839bc6d1916585b3f6a9a 100644
--- a/google_apis/gaia/gaia_auth_fetcher.cc
+++ b/google_apis/gaia/gaia_auth_fetcher.cc
@@ -26,6 +26,7 @@
 #include "net/http/http_response_headers.h"
 #include "net/http/http_status_code.h"
 #include "net/http/http_util.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "net/url_request/url_request_status.h"
@@ -224,9 +225,30 @@ void GaiaAuthFetcher::CreateAndStartGaiaFetcher(const std::string& body,
                                                 const GURL& gaia_gurl,
                                                 int load_flags) {
   DCHECK(!fetch_pending_) << "Tried to fetch two things at once!";
+  net::NetworkTrafficAnnotationTag traffic_annotation =

[msarda, 2017/05/03 09:03:53]

This is a helper method that creates a fetcher that interacts with GAIA (as you
can see the body, the headers, the urls and the load flags are configurable). It
is not clear to me if this needs to be annotated here with somethign generic or
if it should be an argument passed to this function. 

As you can see here
https://cs.chromium.org/chromium/src/google_apis/gaia/gaia_auth_fetcher.h?typ...
,this method is protected, and is called from 13 different places in this file.

[Ramin Halavati, 2017/05/03 09:45:06]

OK, do you that suggest I add annotation as an input to
CreateAndStartGaiaFetcher function, or constructor of GaiaAuthFetcher?

[msarda, 2017/05/10 09:01:54]

If we need to annotate each call differently, then it must be passed in the
method.

However, it is not clear to me in what detail we need to go for each of these
fetchers (can we aggregate the ones that are part of the same flow even though
they fetch different things?).

[Ramin Halavati, 2017/05/10 09:25:20]

Can we can have a generic one here?
If we decide to separate them in future, we can use (the not landed yet) new
annotation definition functions which let part of annotation be defined in
another function and part of it here.

+      net::DefineNetworkTrafficAnnotation("...", R"(
+        semantics {
+          sender: "..."
+          description: "..."
+          trigger: "..."
+          data: "..."
+          destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER
+        }
+        policy {
+          cookies_allowed: false/true
+          cookies_store: "..."
+          setting: "..."
+          chrome_policy {
+            [POLICY_NAME] {
+              policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+              [POLICY_NAME]: ... //(value to disable it)
+            }
+          }
+          policy_exception_justification: "..."
+        })");
   fetcher_ = net::URLFetcher::Create(
       0, gaia_gurl, body.empty() ? net::URLFetcher::GET : net::URLFetcher::POST,
-      this);
+      this, traffic_annotation);
   fetcher_->SetRequestContext(getter_);
   fetcher_->SetUploadData("application/x-www-form-urlencoded", body);
   gaia::MarkURLFetcherAsGaia(fetcher_.get());