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

Side by Side Diff: extensions/browser/api/web_request/web_request_time_tracker.cc

Issue 2479093002: Migrate more files to histogram_macros.h from histogram.h. (Closed)
Patch Set: Created 4 years, 1 month 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
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 "extensions/browser/api/web_request/web_request_time_tracker.h" 5 #include "extensions/browser/api/web_request/web_request_time_tracker.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/metrics/histogram.h" 9 #include "base/metrics/histogram_macros.h"
10 #include "extensions/browser/warning_set.h" 10 #include "extensions/browser/warning_set.h"
11 11
12 12
13 // TODO(mpcomplete): tweak all these constants. 13 // TODO(mpcomplete): tweak all these constants.
14 namespace { 14 namespace {
15 // The number of requests we keep track of at a time. 15 // The number of requests we keep track of at a time.
16 const size_t kMaxRequestsLogged = 100u; 16 const size_t kMaxRequestsLogged = 100u;
17 17
18 // If a request completes faster than this amount (in ms), then we ignore it. 18 // If a request completes faster than this amount (in ms), then we ignore it.
19 // Any delays on such a request was negligible. 19 // Any delays on such a request was negligible.
(...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
244 // When a request is redirected, we have no way of knowing how long the 244 // When a request is redirected, we have no way of knowing how long the
245 // request would have taken, so we can't say how much an extension slowed 245 // request would have taken, so we can't say how much an extension slowed
246 // down this request. Just ignore it. 246 // down this request. Just ignore it.
247 request_time_logs_.erase(request_id); 247 request_time_logs_.erase(request_id);
248 } 248 }
249 249
250 void ExtensionWebRequestTimeTracker::SetDelegate( 250 void ExtensionWebRequestTimeTracker::SetDelegate(
251 ExtensionWebRequestTimeTrackerDelegate* delegate) { 251 ExtensionWebRequestTimeTrackerDelegate* delegate) {
252 delegate_.reset(delegate); 252 delegate_.reset(delegate);
253 } 253 }
OLDNEW
« no previous file with comments | « extensions/browser/api/web_request/web_request_api.cc ('k') | extensions/browser/content_hash_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698