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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: chrome/browser/prerender/prerender_util_unittest.cc

Issue 425653002: content: ResourceType cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: REBASE Created 6 years, 4 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 | « chrome/browser/prerender/prerender_util.cc ('k') | chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h » ('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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "chrome/browser/prerender/prerender_util.h" 5 #include "chrome/browser/prerender/prerender_util.h"
6 6
7 #include "base/metrics/histogram.h" 7 #include "base/metrics/histogram.h"
8 #include "base/metrics/histogram_samples.h" 8 #include "base/metrics/histogram_samples.h"
9 #include "base/metrics/statistics_recorder.h" 9 #include "base/metrics/statistics_recorder.h"
10 #include "net/http/http_response_headers.h" 10 #include "net/http/http_response_headers.h"
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 int num_pss = 0; 95 int num_pss = 0;
96 int num_other = 0; 96 int num_other = 0;
97 int num_bucket_1 = 0; // unrecognized format/value bucket 97 int num_bucket_1 = 0; // unrecognized format/value bucket
98 int num_bucket_30 = 0; // 1.2.24.1 bucket 98 int num_bucket_30 = 0; // 1.2.24.1 bucket
99 int num_bucket_33 = 0; // 1.3.25.2 bucket 99 int num_bucket_33 = 0; // 1.3.25.2 bucket
100 100
101 scoped_ptr<base::HistogramSamples> server_samples; 101 scoped_ptr<base::HistogramSamples> server_samples;
102 scoped_ptr<base::HistogramSamples> version_samples; 102 scoped_ptr<base::HistogramSamples> version_samples;
103 103
104 // No PageSpeed header. The VersionCounts histogram isn't created yet. 104 // No PageSpeed header. The VersionCounts histogram isn't created yet.
105 GatherPagespeedData(ResourceType::MAIN_FRAME, url, headers.get()); 105 GatherPagespeedData(content::RESOURCE_TYPE_MAIN_FRAME, url, headers.get());
106 base::HistogramBase* server_histogram = 106 base::HistogramBase* server_histogram =
107 base::StatisticsRecorder::FindHistogram( 107 base::StatisticsRecorder::FindHistogram(
108 "Prerender.PagespeedHeader.ServerCounts"); 108 "Prerender.PagespeedHeader.ServerCounts");
109 ASSERT_TRUE(server_histogram != NULL); 109 ASSERT_TRUE(server_histogram != NULL);
110 ASSERT_TRUE(NULL == base::StatisticsRecorder::FindHistogram( 110 ASSERT_TRUE(NULL == base::StatisticsRecorder::FindHistogram(
111 "Prerender.PagespeedHeader.VersionCounts")); 111 "Prerender.PagespeedHeader.VersionCounts"));
112 112
113 server_samples = server_histogram->SnapshotSamples(); 113 server_samples = server_histogram->SnapshotSamples();
114 EXPECT_EQ(++num_responses, server_samples->GetCount(0)); 114 EXPECT_EQ(++num_responses, server_samples->GetCount(0));
115 EXPECT_EQ( num_mps, server_samples->GetCount(1)); 115 EXPECT_EQ( num_mps, server_samples->GetCount(1));
116 EXPECT_EQ( num_ngx, server_samples->GetCount(2)); 116 EXPECT_EQ( num_ngx, server_samples->GetCount(2));
117 EXPECT_EQ( num_pss, server_samples->GetCount(3)); 117 EXPECT_EQ( num_pss, server_samples->GetCount(3));
118 EXPECT_EQ( num_other, server_samples->GetCount(4)); 118 EXPECT_EQ( num_other, server_samples->GetCount(4));
119 119
120 // X-Mod-Pagespeed header in expected format. VersionCounts now exists. 120 // X-Mod-Pagespeed header in expected format. VersionCounts now exists.
121 headers->AddHeader("X-Mod-Pagespeed: 1.2.24.1-2300"); 121 headers->AddHeader("X-Mod-Pagespeed: 1.2.24.1-2300");
122 GatherPagespeedData(ResourceType::MAIN_FRAME, url, headers.get()); 122 GatherPagespeedData(content::RESOURCE_TYPE_MAIN_FRAME, url, headers.get());
123 base::HistogramBase* version_histogram = 123 base::HistogramBase* version_histogram =
124 base::StatisticsRecorder::FindHistogram( 124 base::StatisticsRecorder::FindHistogram(
125 "Prerender.PagespeedHeader.VersionCounts"); 125 "Prerender.PagespeedHeader.VersionCounts");
126 ASSERT_TRUE(version_histogram != NULL); 126 ASSERT_TRUE(version_histogram != NULL);
127 server_samples = server_histogram->SnapshotSamples(); 127 server_samples = server_histogram->SnapshotSamples();
128 version_samples = version_histogram->SnapshotSamples(); 128 version_samples = version_histogram->SnapshotSamples();
129 EXPECT_EQ(++num_responses, server_samples->GetCount(0)); 129 EXPECT_EQ(++num_responses, server_samples->GetCount(0));
130 EXPECT_EQ(++num_mps, server_samples->GetCount(1)); 130 EXPECT_EQ(++num_mps, server_samples->GetCount(1));
131 EXPECT_EQ( num_ngx, server_samples->GetCount(2)); 131 EXPECT_EQ( num_ngx, server_samples->GetCount(2));
132 EXPECT_EQ( num_pss, server_samples->GetCount(3)); 132 EXPECT_EQ( num_pss, server_samples->GetCount(3));
133 EXPECT_EQ( num_other, server_samples->GetCount(4)); 133 EXPECT_EQ( num_other, server_samples->GetCount(4));
134 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1)); 134 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1));
135 EXPECT_EQ(++num_bucket_30, version_samples->GetCount(30)); // +1 for #30 135 EXPECT_EQ(++num_bucket_30, version_samples->GetCount(30)); // +1 for #30
136 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33)); 136 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33));
137 headers->RemoveHeader("X-Mod-Pagespeed"); 137 headers->RemoveHeader("X-Mod-Pagespeed");
138 138
139 // X-Mod-Pagespeed header in unexpected format. 139 // X-Mod-Pagespeed header in unexpected format.
140 headers->AddHeader("X-Mod-Pagespeed: Powered By PageSpeed!"); 140 headers->AddHeader("X-Mod-Pagespeed: Powered By PageSpeed!");
141 GatherPagespeedData(ResourceType::MAIN_FRAME, url, headers.get()); 141 GatherPagespeedData(content::RESOURCE_TYPE_MAIN_FRAME, url, headers.get());
142 server_samples = server_histogram->SnapshotSamples(); 142 server_samples = server_histogram->SnapshotSamples();
143 version_samples = version_histogram->SnapshotSamples(); 143 version_samples = version_histogram->SnapshotSamples();
144 EXPECT_EQ(++num_responses, server_samples->GetCount(0)); 144 EXPECT_EQ(++num_responses, server_samples->GetCount(0));
145 EXPECT_EQ(++num_mps, server_samples->GetCount(1)); 145 EXPECT_EQ(++num_mps, server_samples->GetCount(1));
146 EXPECT_EQ( num_ngx, server_samples->GetCount(2)); 146 EXPECT_EQ( num_ngx, server_samples->GetCount(2));
147 EXPECT_EQ( num_pss, server_samples->GetCount(3)); 147 EXPECT_EQ( num_pss, server_samples->GetCount(3));
148 EXPECT_EQ( num_other, server_samples->GetCount(4)); 148 EXPECT_EQ( num_other, server_samples->GetCount(4));
149 EXPECT_EQ(++num_bucket_1, version_samples->GetCount(1)); // +1 for 'huh?' 149 EXPECT_EQ(++num_bucket_1, version_samples->GetCount(1)); // +1 for 'huh?'
150 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30)); 150 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30));
151 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33)); 151 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33));
152 headers->RemoveHeader("X-Mod-Pagespeed"); 152 headers->RemoveHeader("X-Mod-Pagespeed");
153 153
154 // X-Page-Speed header in mod_pagespeed format (so ngx_pagespeed). 154 // X-Page-Speed header in mod_pagespeed format (so ngx_pagespeed).
155 headers->AddHeader("X-Page-Speed: 1.3.25.2-2530"); 155 headers->AddHeader("X-Page-Speed: 1.3.25.2-2530");
156 GatherPagespeedData(ResourceType::MAIN_FRAME, url, headers.get()); 156 GatherPagespeedData(content::RESOURCE_TYPE_MAIN_FRAME, url, headers.get());
157 server_samples = server_histogram->SnapshotSamples(); 157 server_samples = server_histogram->SnapshotSamples();
158 version_samples = version_histogram->SnapshotSamples(); 158 version_samples = version_histogram->SnapshotSamples();
159 EXPECT_EQ(++num_responses, server_samples->GetCount(0)); 159 EXPECT_EQ(++num_responses, server_samples->GetCount(0));
160 EXPECT_EQ( num_mps, server_samples->GetCount(1)); 160 EXPECT_EQ( num_mps, server_samples->GetCount(1));
161 EXPECT_EQ(++num_ngx, server_samples->GetCount(2)); 161 EXPECT_EQ(++num_ngx, server_samples->GetCount(2));
162 EXPECT_EQ( num_pss, server_samples->GetCount(3)); 162 EXPECT_EQ( num_pss, server_samples->GetCount(3));
163 EXPECT_EQ( num_other, server_samples->GetCount(4)); 163 EXPECT_EQ( num_other, server_samples->GetCount(4));
164 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1)); 164 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1));
165 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30)); 165 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30));
166 EXPECT_EQ(++num_bucket_33, version_samples->GetCount(33)); // +1 for #33 166 EXPECT_EQ(++num_bucket_33, version_samples->GetCount(33)); // +1 for #33
167 headers->RemoveHeader("X-Page-Speed"); 167 headers->RemoveHeader("X-Page-Speed");
168 168
169 // X-Page-Speed header in PageSpeed Service format. 169 // X-Page-Speed header in PageSpeed Service format.
170 headers->AddHeader("X-Page-Speed: 97_4_bo"); 170 headers->AddHeader("X-Page-Speed: 97_4_bo");
171 GatherPagespeedData(ResourceType::MAIN_FRAME, url, headers.get()); 171 GatherPagespeedData(content::RESOURCE_TYPE_MAIN_FRAME, url, headers.get());
172 server_samples = server_histogram->SnapshotSamples(); 172 server_samples = server_histogram->SnapshotSamples();
173 version_samples = version_histogram->SnapshotSamples(); 173 version_samples = version_histogram->SnapshotSamples();
174 EXPECT_EQ(++num_responses, server_samples->GetCount(0)); 174 EXPECT_EQ(++num_responses, server_samples->GetCount(0));
175 EXPECT_EQ( num_mps, server_samples->GetCount(1)); // no change 175 EXPECT_EQ( num_mps, server_samples->GetCount(1)); // no change
176 EXPECT_EQ( num_ngx, server_samples->GetCount(2)); 176 EXPECT_EQ( num_ngx, server_samples->GetCount(2));
177 EXPECT_EQ(++num_pss, server_samples->GetCount(3)); // +1 for PSS 177 EXPECT_EQ(++num_pss, server_samples->GetCount(3)); // +1 for PSS
178 EXPECT_EQ( num_other, server_samples->GetCount(4)); 178 EXPECT_EQ( num_other, server_samples->GetCount(4));
179 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1)); 179 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1));
180 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30)); 180 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30));
181 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33)); 181 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33));
182 headers->RemoveHeader("X-Page-Speed"); 182 headers->RemoveHeader("X-Page-Speed");
183 183
184 // X-Page-Speed header in an unrecognized format (IISpeed in this case). 184 // X-Page-Speed header in an unrecognized format (IISpeed in this case).
185 headers->AddHeader("X-Page-Speed: 1.0PS1.2-20130615"); 185 headers->AddHeader("X-Page-Speed: 1.0PS1.2-20130615");
186 GatherPagespeedData(ResourceType::MAIN_FRAME, url, headers.get()); 186 GatherPagespeedData(content::RESOURCE_TYPE_MAIN_FRAME, url, headers.get());
187 server_samples = server_histogram->SnapshotSamples(); 187 server_samples = server_histogram->SnapshotSamples();
188 version_samples = version_histogram->SnapshotSamples(); 188 version_samples = version_histogram->SnapshotSamples();
189 EXPECT_EQ(++num_responses, server_samples->GetCount(0)); 189 EXPECT_EQ(++num_responses, server_samples->GetCount(0));
190 EXPECT_EQ( num_mps, server_samples->GetCount(1)); // no change 190 EXPECT_EQ( num_mps, server_samples->GetCount(1)); // no change
191 EXPECT_EQ( num_pss, server_samples->GetCount(3)); 191 EXPECT_EQ( num_pss, server_samples->GetCount(3));
192 EXPECT_EQ(++num_other, server_samples->GetCount(4)); // +1 for 'other' 192 EXPECT_EQ(++num_other, server_samples->GetCount(4)); // +1 for 'other'
193 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1)); 193 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1));
194 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30)); 194 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30));
195 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33)); 195 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33));
196 196
197 // Not a main frame => not counted at all. 197 // Not a main frame => not counted at all.
198 GatherPagespeedData(ResourceType::SUB_FRAME, url, headers.get()); 198 GatherPagespeedData(content::RESOURCE_TYPE_SUB_FRAME, url, headers.get());
199 server_samples = server_histogram->SnapshotSamples(); 199 server_samples = server_histogram->SnapshotSamples();
200 version_samples = version_histogram->SnapshotSamples(); 200 version_samples = version_histogram->SnapshotSamples();
201 EXPECT_EQ( num_responses, server_samples->GetCount(0)); 201 EXPECT_EQ( num_responses, server_samples->GetCount(0));
202 EXPECT_EQ( num_mps, server_samples->GetCount(1)); 202 EXPECT_EQ( num_mps, server_samples->GetCount(1));
203 EXPECT_EQ( num_ngx, server_samples->GetCount(2)); 203 EXPECT_EQ( num_ngx, server_samples->GetCount(2));
204 EXPECT_EQ( num_pss, server_samples->GetCount(3)); 204 EXPECT_EQ( num_pss, server_samples->GetCount(3));
205 EXPECT_EQ( num_other, server_samples->GetCount(4)); 205 EXPECT_EQ( num_other, server_samples->GetCount(4));
206 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1)); 206 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1));
207 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30)); 207 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30));
208 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33)); 208 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33));
209 209
210 // Not a http/https URL => not counted at all. 210 // Not a http/https URL => not counted at all.
211 GURL data_url("data:image/png;base64,yadda yadda=="); 211 GURL data_url("data:image/png;base64,yadda yadda==");
212 GatherPagespeedData(ResourceType::MAIN_FRAME, data_url, headers.get()); 212 GatherPagespeedData(
213 content::RESOURCE_TYPE_MAIN_FRAME, data_url, headers.get());
213 server_samples = server_histogram->SnapshotSamples(); 214 server_samples = server_histogram->SnapshotSamples();
214 version_samples = version_histogram->SnapshotSamples(); 215 version_samples = version_histogram->SnapshotSamples();
215 EXPECT_EQ( num_responses, server_samples->GetCount(0)); 216 EXPECT_EQ( num_responses, server_samples->GetCount(0));
216 EXPECT_EQ( num_mps, server_samples->GetCount(1)); 217 EXPECT_EQ( num_mps, server_samples->GetCount(1));
217 EXPECT_EQ( num_ngx, server_samples->GetCount(2)); 218 EXPECT_EQ( num_ngx, server_samples->GetCount(2));
218 EXPECT_EQ( num_pss, server_samples->GetCount(3)); 219 EXPECT_EQ( num_pss, server_samples->GetCount(3));
219 EXPECT_EQ( num_other, server_samples->GetCount(4)); 220 EXPECT_EQ( num_other, server_samples->GetCount(4));
220 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1)); 221 EXPECT_EQ( num_bucket_1, version_samples->GetCount(1));
221 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30)); 222 EXPECT_EQ( num_bucket_30, version_samples->GetCount(30));
222 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33)); 223 EXPECT_EQ( num_bucket_33, version_samples->GetCount(33));
223 224
224 headers->RemoveHeader("X-Page-Speed"); 225 headers->RemoveHeader("X-Page-Speed");
225 } 226 }
226 227
227 } // namespace prerender 228 } // namespace prerender
OLDNEW
« no previous file with comments | « chrome/browser/prerender/prerender_util.cc ('k') | chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698