Network traffic annotation added to sync.

components/sync/engine_impl/attachments/attachment_uploader_impl.cc

 // Copyright 2014 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 "components/sync/engine_impl/attachments/attachment_uploader_impl.h"
 
 #include <utility>
 #include <vector>
 
 #include "base/base64.h"
 #include "base/base64url.h"
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/memory/weak_ptr.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/stringprintf.h"
 #include "base/sys_byteorder.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "components/data_use_measurement/core/data_use_user_data.h"
 #include "components/sync/model/attachments/attachment.h"
 #include "components/sync/protocol/sync.pb.h"
 #include "google_apis/gaia/gaia_constants.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"
 #include "net/url_request/url_request_status.h"
 
 namespace {
 
 const char kContentType[] = "application/octet-stream";
 const char kAttachments[] = "attachments/";
 const char kSyncStoreBirthday[] = "X-Sync-Store-Birthday";
 const char kSyncDataTypeId[] = "X-Sync-Data-Type-Id";
@@ skipping 170 matching lines @@
     const std::string& access_token,
     const base::Time& expiration_time) {
   DCHECK(CalledOnValidThread());
   if (is_stopped_) {
     return;
   }
 
   DCHECK_EQ(access_token_request_.get(), request);
   access_token_request_.reset();
   access_token_ = access_token;
-  fetcher_ = net::URLFetcher::Create(upload_url_, net::URLFetcher::POST, this);
+  net::NetworkTrafficAnnotationTag traffic_annotation =

[Nicolas Zea, 2017/03/03 22:39:56]

same as the http bridge

[Ramin Halavati, 2017/03/06 06:55:45]

Done.

+      net::DefineNetworkTrafficAnnotation("...", R"(
+        semantics {
+          sender: "..."
+          description: "..."
+          trigger: "..."
+          data: "..."
+          destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER
+        }
+        policy {
+          cookies_allowed: false/true
+          cookies_store: "..."
+          setting: "..."
+          policy {
+            [POLICY_NAME] {
+              policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+              value: ...
+            }
+          }
+          policy_exception_justification: "..."
+        })");
+  fetcher_ = net::URLFetcher::Create(upload_url_, net::URLFetcher::POST, this,
+                                     traffic_annotation);
   ConfigureURLFetcherCommon(fetcher_.get(), access_token_, raw_store_birthday_,
                             model_type_, url_request_context_getter_.get());
   data_use_measurement::DataUseUserData::AttachToFetcher(
       fetcher_.get(), data_use_measurement::DataUseUserData::SYNC);
   const uint32_t crc32c = attachment_.GetCrc32c();
   fetcher_->AddExtraRequestHeader(base::StringPrintf(
       "X-Goog-Hash: crc32c=%s", FormatCrc32cHash(crc32c).c_str()));
 
   // TODO(maniscalco): Is there a better way?  Copying the attachment data into
   // a string feels wrong given how large attachments may be (several MBs).  If
@@ skipping 163 matching lines @@
       "%s: %s", kSyncStoreBirthday, encoded_store_birthday.c_str()));
 
   // Use field number to pass ModelType because it's stable and we have server
   // code to decode it.
   const int field_number = GetSpecificsFieldNumberFromModelType(model_type);
   fetcher->AddExtraRequestHeader(
       base::StringPrintf("%s: %d", kSyncDataTypeId, field_number));
 }
 
 }  // namespace syncer