Pass time since profile was loaded and request count on each ListAccount request to Gaia.

google_apis/gaia/gaia_auth_fetcher.cc

 // Copyright (c) 2012 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 "google_apis/gaia/gaia_auth_fetcher.h"
 
 #include <string>
 #include <utility>
 #include <vector>
 
@@ skipping 631 matching lines @@
                                       const std::string& service) {
   DCHECK(!fetch_pending_) << "Tried to fetch two things at once!";
 
   request_body_ = MakeOAuthLoginBody(service, source_);
   std::string authentication_header =
       base::StringPrintf(kOAuth2BearerHeaderFormat, access_token.c_str());
   CreateAndStartGaiaFetcher(request_body_, authentication_header,
                             oauth_login_gurl_, net::LOAD_NORMAL);
 }
 
-void GaiaAuthFetcher::StartListAccounts() {
+void GaiaAuthFetcher::StartListAccounts(const base::Time& profile_load_time) {
   DCHECK(!fetch_pending_) << "Tried to fetch two things at once!";
 
+  // Report the time since the profile was loaded and the number of this request
+  // in order to debug channel ID issues observed on Gaia.
+  static int32_t list_accounts_request_counter = 0;
+  std::string extra_headers = base::StringPrintf(
+      "Origin: %s \r\n RequestCount: %d \r\n TimeSinceProfileLoad: %ld",
+      "https://www.google.com", list_accounts_request_counter++,
+      (base::Time::Now() - profile_load_time).InMilliseconds());
   CreateAndStartGaiaFetcher(" ",  // To force an HTTP POST.
-                            "Origin: https://www.google.com",
-                            list_accounts_gurl_, net::LOAD_NORMAL);
+                            extra_headers, list_accounts_gurl_,

[vanupam, 2017/02/15 03:08:28]

list_accounts_gurl_ is assigned in the constructor via
ListAccountsURLWithSource(source) which lets a source=xxx url param be set.

Since this is a debugging-only change (hopefully, it will be reverted in the
near term), instead of adding new headers (which will require server side
changes to propagate into debug logs) can we append e.g. _counter_loadtime to
the source param; (The source param value has no semantics in Gaia.)

[msarda, 2017/03/01 10:26:36]

Done.

+                            net::LOAD_NORMAL);
 }
 
 void GaiaAuthFetcher::StartLogOut() {
   DCHECK(!fetch_pending_) << "Tried to fetch two things at once!";
 
   CreateAndStartGaiaFetcher(std::string(), logout_headers_, logout_gurl_,
                             net::LOAD_NORMAL);
 }
 
 void GaiaAuthFetcher::StartGetCheckConnectionInfo() {
@@ skipping 345 matching lines @@
   return alleged_error.find(kSecondFactor) !=
       std::string::npos;
 }
 
 // static
 bool GaiaAuthFetcher::IsWebLoginRequiredSuccess(
     const std::string& alleged_error) {
   return alleged_error.find(kWebLoginRequired) !=
       std::string::npos;
 }