Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(183)

Side by Side Diff: net/url_request/url_request_context.cc

Issue 2897143003: Use NetworkTrafficAnnotation in URLRequestContext UMA logging (Closed)
Patch Set: Address Alexei's comments Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | tools/metrics/histograms/enums.xml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/url_request/url_request_context.h" 5 #include "net/url_request/url_request_context.h"
6 6
7 #include <inttypes.h> 7 #include <inttypes.h>
8 8
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/debug/alias.h" 10 #include "base/debug/alias.h"
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 void URLRequestContext::set_cookie_store(CookieStore* cookie_store) { 120 void URLRequestContext::set_cookie_store(CookieStore* cookie_store) {
121 cookie_store_ = cookie_store; 121 cookie_store_ = cookie_store;
122 } 122 }
123 123
124 void URLRequestContext::InsertURLRequest(const URLRequest* request) const { 124 void URLRequestContext::InsertURLRequest(const URLRequest* request) const {
125 url_requests_.insert(request); 125 url_requests_.insert(request);
126 if (url_requests_.size() > largest_outstanding_requests_count_seen_) { 126 if (url_requests_.size() > largest_outstanding_requests_count_seen_) {
127 largest_outstanding_requests_count_seen_ = url_requests_.size(); 127 largest_outstanding_requests_count_seen_ = url_requests_.size();
128 UMA_HISTOGRAM_COUNTS_1M("Net.URLRequestContext.OutstandingRequests", 128 UMA_HISTOGRAM_COUNTS_1M("Net.URLRequestContext.OutstandingRequests",
129 largest_outstanding_requests_count_seen_); 129 largest_outstanding_requests_count_seen_);
130 UMA_HISTOGRAM_SPARSE_SLOWLY(
131 "Net.URLRequestContext.OutstandingRequests.Type",
132 request->traffic_annotation().unique_id_hash_code);
130 } 133 }
131 } 134 }
132 135
133 void URLRequestContext::RemoveURLRequest(const URLRequest* request) const { 136 void URLRequestContext::RemoveURLRequest(const URLRequest* request) const {
134 DCHECK_EQ(1u, url_requests_.count(request)); 137 DCHECK_EQ(1u, url_requests_.count(request));
135 url_requests_.erase(request); 138 url_requests_.erase(request);
136 } 139 }
137 140
138 void URLRequestContext::AssertNoURLRequests() const { 141 void URLRequestContext::AssertNoURLRequests() const {
139 int num_requests = url_requests_.size(); 142 int num_requests = url_requests_.size();
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 HttpCache* http_cache = transaction_factory->GetCache(); 179 HttpCache* http_cache = transaction_factory->GetCache();
177 if (http_cache) 180 if (http_cache)
178 http_cache->DumpMemoryStats(pmd, dump->absolute_name()); 181 http_cache->DumpMemoryStats(pmd, dump->absolute_name());
179 } 182 }
180 if (sdch_manager_) 183 if (sdch_manager_)
181 sdch_manager_->DumpMemoryStats(pmd, dump_name); 184 sdch_manager_->DumpMemoryStats(pmd, dump_name);
182 return true; 185 return true;
183 } 186 }
184 187
185 } // namespace net 188 } // namespace net
OLDNEW
« no previous file with comments | « no previous file | tools/metrics/histograms/enums.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698