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

Unified Diff: chrome/browser/prerender/prerender_manager.h

Issue 2302433002: [NoStatePrefetch] Track redirects in UMA (Closed)
Patch Set: Rebase Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/prerender/prerender_histograms.cc ('k') | chrome/browser/prerender/prerender_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_manager.h
diff --git a/chrome/browser/prerender/prerender_manager.h b/chrome/browser/prerender/prerender_manager.h
index 47ad9e50e375a7ea7ac01645aa5589c34cf14ad9..ec8d4039d42046334dba90e244ebf1eacb197867 100644
--- a/chrome/browser/prerender/prerender_manager.h
+++ b/chrome/browser/prerender/prerender_manager.h
@@ -199,10 +199,18 @@ class PrerenderManager : public base::SupportsWeakPtr<PrerenderManager>,
double fraction_plt_elapsed_at_swap_in,
const GURL& url);
- // Called when a resource has been loaded by NoStatePrefetch.
- void RecordResourcePrefetch(Origin origin,
- bool is_main_resource,
- bool is_no_store);
+ // Called when a NoStatePrefetch request has received a response (including
+ // redirects). May be called several times per resource, in case of redirects.
+ void RecordPrefetchResponseReceived(Origin origin,
+ bool is_main_resource,
+ bool is_redirect,
+ bool is_no_store);
+
+ // Called when a NoStatePrefetch resource has been loaded. This is called only
+ // once per resource, when all redirects have been resolved.
+ void RecordPrefetchRedirectCount(Origin origin,
+ bool is_main_resource,
+ int redirect_count);
static PrerenderManagerMode GetMode();
static void SetMode(PrerenderManagerMode mode);
« no previous file with comments | « chrome/browser/prerender/prerender_histograms.cc ('k') | chrome/browser/prerender/prerender_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698