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

Side by Side Diff: net/disk_cache/v3/histogram_macros.h

Issue 140893013: Clean up dependencies for disk_cache histogram_macros.h. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix fat finger Created 6 years, 10 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 | Annotate | Revision Log
« no previous file with comments | « net/disk_cache/v3/eviction_v3.cc ('k') | net/net.gyp » ('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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 // This file contains macros to simplify histogram reporting from the disk 5 // This file contains macros to simplify histogram reporting from the disk
6 // cache. The main issue is that we want to have separate histograms for each 6 // cache. The main issue is that we want to have separate histograms for each
7 // type of cache (regular vs. media, etc), without adding the complexity of 7 // type of cache (regular vs. media, etc), without adding the complexity of
8 // keeping track of a potentially large number of histogram objects that have to 8 // keeping track of a potentially large number of histogram objects that have to
9 // survive the backend object that created them. 9 // survive the backend object that created them.
10 10
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 (base::Time::Now() - initial_time).InHours()) 77 (base::Time::Now() - initial_time).InHours())
78 78
79 // HISTOGRAM_AGE_MS will collect time elapsed since |initial_time|, with the 79 // HISTOGRAM_AGE_MS will collect time elapsed since |initial_time|, with the
80 // normal resolution of the UMA_HISTOGRAM_TIMES. 80 // normal resolution of the UMA_HISTOGRAM_TIMES.
81 #define CACHE_HISTOGRAM_AGE_MS(name, initial_time)\ 81 #define CACHE_HISTOGRAM_AGE_MS(name, initial_time)\
82 CACHE_HISTOGRAM_TIMES(name, base::TimeTicks::Now() - initial_time) 82 CACHE_HISTOGRAM_TIMES(name, base::TimeTicks::Now() - initial_time)
83 83
84 #define CACHE_HISTOGRAM_CACHE_ERROR(name, sample) \ 84 #define CACHE_HISTOGRAM_CACHE_ERROR(name, sample) \
85 CACHE_HISTOGRAM_ENUMERATION(name, sample, 50) 85 CACHE_HISTOGRAM_ENUMERATION(name, sample, 50)
86 86
87 #ifdef NET_DISK_CACHE_BACKEND_IMPL_CC_
88 #define BACKEND_OBJ this
89 #else
90 #define BACKEND_OBJ backend_
91 #endif
92
93 // Generates a UMA histogram of the given type, generating the proper name for 87 // Generates a UMA histogram of the given type, generating the proper name for
94 // it (asking backend_->HistogramName), and adding the provided sample. 88 // it (asking backend_->HistogramName), and adding the provided sample.
95 // For example, to generate a regualar UMA_HISTOGRAM_COUNTS, this macro would 89 // For example, to generate a regualar UMA_HISTOGRAM_COUNTS, this macro would
96 // be used as: 90 // be used as:
97 // CACHE_UMA(COUNTS, "MyName", 20); 91 // CACHE_UMA(COUNTS, "MyName", 20);
98 // which may translate to: 92 // which may translate to:
99 // UMA_HISTOGRAM_COUNTS("DiskCache3.MyName_AppCache", 20); 93 // UMA_HISTOGRAM_COUNTS("DiskCache3.MyName_AppCache", 20);
100 // 94 //
101 #define CACHE_UMA(type, name, sample) {\ 95 #define CACHE_UMA(type, name, sample) {\
102 const std::string my_name = BACKEND_OBJ->HistogramName(name);\ 96 const std::string my_name =\
103 switch (BACKEND_OBJ->cache_type()) {\ 97 CACHE_UMA_BACKEND_IMPL_OBJ->HistogramName(name);\
98 switch (CACHE_UMA_BACKEND_IMPL_OBJ->cache_type()) {\
104 case net::DISK_CACHE:\ 99 case net::DISK_CACHE:\
105 CACHE_HISTOGRAM_##type(my_name.data(), sample);\ 100 CACHE_HISTOGRAM_##type(my_name.data(), sample);\
106 break;\ 101 break;\
107 case net::MEDIA_CACHE:\ 102 case net::MEDIA_CACHE:\
108 CACHE_HISTOGRAM_##type(my_name.data(), sample);\ 103 CACHE_HISTOGRAM_##type(my_name.data(), sample);\
109 break;\ 104 break;\
110 case net::APP_CACHE:\ 105 case net::APP_CACHE:\
111 CACHE_HISTOGRAM_##type(my_name.data(), sample);\ 106 CACHE_HISTOGRAM_##type(my_name.data(), sample);\
112 break;\ 107 break;\
113 case net::SHADER_CACHE:\ 108 case net::SHADER_CACHE:\
114 CACHE_HISTOGRAM_##type(my_name.data(), sample);\ 109 CACHE_HISTOGRAM_##type(my_name.data(), sample);\
115 break;\ 110 break;\
116 case net::PNACL_CACHE:\ 111 case net::PNACL_CACHE:\
117 CACHE_HISTOGRAM_##type(my_name.data(), sample);\ 112 CACHE_HISTOGRAM_##type(my_name.data(), sample);\
118 break;\ 113 break;\
119 default:\ 114 default:\
120 break;\ 115 break;\
121 }\ 116 }\
122 } 117 }
123 118
124 #endif // NET_DISK_CACHE_V3_HISTOGRAM_MACROS_H_ 119 #endif // NET_DISK_CACHE_V3_HISTOGRAM_MACROS_H_
OLDNEW
« no previous file with comments | « net/disk_cache/v3/eviction_v3.cc ('k') | net/net.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698