Network traffic annotation added to SDCH Dictionary Fetcher.

net/url_request/sdch_dictionary_fetcher.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 "net/url_request/sdch_dictionary_fetcher.h"
 
 #include <stdint.h>
 #include <queue>
 #include <set>
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "net/base/io_buffer.h"
 #include "net/base/load_flags.h"
 #include "net/base/sdch_net_log_params.h"
 #include "net/http/http_response_headers.h"
 #include "net/log/net_log_event_type.h"
 #include "net/log/net_log_with_source.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/redirect_info.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_status.h"
 #include "net/url_request/url_request_throttler_manager.h"
 
 namespace net {
 
 namespace {
 
 const int kBufferSize = 4096;
@@ skipping 239 matching lines @@
   if (fetch_queue_->IsEmpty() || current_request_.get()) {
     next_state_ = STATE_NONE;
     return OK;
   }
 
   next_state_ = STATE_SEND_REQUEST_PENDING;
 
   FetchInfo info;
   bool success = fetch_queue_->Pop(&info);
   DCHECK(success);
-  current_request_ = context_->CreateRequest(info.url, IDLE, this);
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("sdch_dictionary_fetch", R"(
+        semantics {
+          sender: "SDCH"
+          description:
+            "The Chrome Network Stack can use less bandwidth and reduce "

[Randy Smith (Not in Mondays), 2017/05/03 12:58:42]

nit: reduces.  (My mistake.)

[Ramin Halavati, 2017/05/04 04:36:28]

Done.

+            "request latency if a dictionary shared between client and server "
+            "is used to compress content on the server before sending, and "
+            "decompress content on the client after reception. This request is "
+            "fetching such a dictionary; it is dispatched when the response to "
+            "a previous URL request indicated that there was a dictionary that "
+            "the client did not currently have in memory, and could be used "
+            "for compression."
+          trigger:
+            "A response to a previous URL request indicated that this URL "
+            "contained a dictionary that could be used to compress other URL "
+            "responses."
+          data:
+            "..."
+          destination: WEBSITE
+        }
+        policy {
+          cookies_allowed: false
+          setting:
+            "This feature is disabled in Chrome and can obly be enabled in non-"

[Randy Smith (Not in Mondays), 2017/05/03 12:58:43]

nit: only

[Ramin Halavati, 2017/05/04 04:36:28]

Done.

+            "Chromium embedders."
+          policy_exception_justification:
+            "Not implemented, not part of Chrome."
+        })");
+  current_request_ =
+      context_->CreateRequest(info.url, IDLE, this, traffic_annotation);
   int load_flags = LOAD_DO_NOT_SEND_COOKIES | LOAD_DO_NOT_SAVE_COOKIES;
   if (info.cache_only)
     load_flags |= LOAD_ONLY_FROM_CACHE | LOAD_SKIP_CACHE_VALIDATION;
   current_request_->SetLoadFlags(load_flags);
 
   buffer_ = new IOBuffer(kBufferSize);
   dictionary_.reset(new std::string());
   current_callback_ = info.callback;
 
   current_request_->Start();
@@ skipping 76 matching lines @@
                           current_request_->net_log(),
                           current_request_->was_cached());
   }
 
   ResetRequest();
   next_state_ = STATE_SEND_REQUEST;
   return OK;
 }
 
 }  // namespace net