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

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

Issue 23064011: Consolidate scheme checks into an easy GURL method (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 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
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 "chrome/browser/prerender/prerender_histograms.h" 5 #include "chrome/browser/prerender/prerender_histograms.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/format_macros.h" 9 #include "base/format_macros.h"
10 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
(...skipping 232 matching lines...) Expand 10 before | Expand all | Expand 10 after
243 // ...FirstAfterMissAnyOnly -- pages meeting 243 // ...FirstAfterMissAnyOnly -- pages meeting
244 // FirstAfterMiss but NOT FirstAfterMissNonOverlapping 244 // FirstAfterMiss but NOT FirstAfterMissNonOverlapping
245 // ..FirstAfterMissNonOverlappingOnly -- pages meeting 245 // ..FirstAfterMissNonOverlappingOnly -- pages meeting
246 // FirstAfterMissNonOverlapping but NOT FirstAfterMiss 246 // FirstAfterMissNonOverlapping but NOT FirstAfterMiss
247 247
248 void PrerenderHistograms::RecordPerceivedPageLoadTime( 248 void PrerenderHistograms::RecordPerceivedPageLoadTime(
249 Origin origin, 249 Origin origin,
250 base::TimeDelta perceived_page_load_time, 250 base::TimeDelta perceived_page_load_time,
251 bool was_prerender, 251 bool was_prerender,
252 bool was_complete_prerender, const GURL& url) { 252 bool was_complete_prerender, const GURL& url) {
253 if (!IsWebURL(url)) 253 if (!url.SchemeIsHTTPOrHTTPS())
254 return; 254 return;
255 bool within_window = WithinWindow(); 255 bool within_window = WithinWindow();
256 bool is_google_url = IsGoogleDomain(url); 256 bool is_google_url = IsGoogleDomain(url);
257 RECORD_PLT("PerceivedPLT", perceived_page_load_time); 257 RECORD_PLT("PerceivedPLT", perceived_page_load_time);
258 if (within_window) 258 if (within_window)
259 RECORD_PLT("PerceivedPLTWindowed", perceived_page_load_time); 259 RECORD_PLT("PerceivedPLTWindowed", perceived_page_load_time);
260 if (was_prerender || was_complete_prerender) { 260 if (was_prerender || was_complete_prerender) {
261 if (was_prerender) 261 if (was_prerender)
262 RECORD_PLT("PerceivedPLTMatched", perceived_page_load_time); 262 RECORD_PLT("PerceivedPLTMatched", perceived_page_load_time);
263 if (was_complete_prerender) 263 if (was_complete_prerender)
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
296 } 296 }
297 } 297 }
298 } 298 }
299 299
300 void PrerenderHistograms::RecordPageLoadTimeNotSwappedIn( 300 void PrerenderHistograms::RecordPageLoadTimeNotSwappedIn(
301 Origin origin, 301 Origin origin,
302 base::TimeDelta page_load_time, 302 base::TimeDelta page_load_time,
303 const GURL& url) const { 303 const GURL& url) const {
304 // If the URL to be prerendered is not a http[s] URL, or is a Google URL, 304 // If the URL to be prerendered is not a http[s] URL, or is a Google URL,
305 // do not record. 305 // do not record.
306 if (!IsWebURL(url) || IsGoogleDomain(url)) 306 if (!url.SchemeIsHTTPOrHTTPS() || IsGoogleDomain(url))
307 return; 307 return;
308 RECORD_PLT("PrerenderNotSwappedInPLT", page_load_time); 308 RECORD_PLT("PrerenderNotSwappedInPLT", page_load_time);
309 } 309 }
310 310
311 void PrerenderHistograms::RecordPercentLoadDoneAtSwapin(Origin origin, 311 void PrerenderHistograms::RecordPercentLoadDoneAtSwapin(Origin origin,
312 double fraction) const { 312 double fraction) const {
313 if (fraction < 0.0 || fraction > 1.0) 313 if (fraction < 0.0 || fraction > 1.0)
314 return; 314 return;
315 int percentage = static_cast<int>(fraction * 100); 315 int percentage = static_cast<int>(fraction * 100);
316 if (percentage < 0 || percentage > 100) 316 if (percentage < 0 || percentage > 100)
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
385 return last_experiment_id_; 385 return last_experiment_id_;
386 } 386 }
387 387
388 bool PrerenderHistograms::IsOriginExperimentWash() const { 388 bool PrerenderHistograms::IsOriginExperimentWash() const {
389 if (!WithinWindow()) 389 if (!WithinWindow())
390 return false; 390 return false;
391 return origin_experiment_wash_; 391 return origin_experiment_wash_;
392 } 392 }
393 393
394 } // namespace prerender 394 } // namespace prerender
OLDNEW
« no previous file with comments | « chrome/browser/net/predictor_tab_helper.cc ('k') | chrome/browser/prerender/prerender_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698