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

Unified Diff: chrome/browser/prerender/prerender_contents.cc

Issue 7289020: Updating histograms to allow for experiments & log origin-based histograms. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 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_contents.h ('k') | chrome/browser/prerender/prerender_final_status.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_contents.cc
===================================================================
--- chrome/browser/prerender/prerender_contents.cc (revision 91750)
+++ chrome/browser/prerender/prerender_contents.cc (working copy)
@@ -63,9 +63,9 @@
virtual PrerenderContents* CreatePrerenderContents(
PrerenderManager* prerender_manager, PrerenderTracker* prerender_tracker,
Profile* profile, const GURL& url, const GURL& referrer,
- Origin origin) OVERRIDE {
+ Origin origin, uint8 experiment_id) OVERRIDE {
return new PrerenderContents(prerender_manager, prerender_tracker, profile,
- url, referrer, origin);
+ url, referrer, origin, experiment_id);
}
};
@@ -109,7 +109,8 @@
Profile* profile,
const GURL& url,
const GURL& referrer,
- Origin origin)
+ Origin origin,
+ uint8 experiment_id)
: prerender_manager_(prerender_manager),
prerender_tracker_(prerender_tracker),
prerender_url_(url),
@@ -123,7 +124,8 @@
child_id_(-1),
route_id_(-1),
starting_page_id_(-1),
- origin_(origin) {
+ origin_(origin),
+ experiment_id_(experiment_id) {
DCHECK(prerender_manager != NULL);
}
@@ -294,7 +296,8 @@
// If we haven't even started prerendering, we were just in the control
// group, which means we do not want to record the status.
if (prerendering_has_started())
- RecordFinalStatus(origin_, final_status_);
+ prerender_manager_->RecordFinalStatus(origin_, experiment_id_,
+ final_status_);
if (child_id_ != -1 && route_id_ != -1)
prerender_tracker_->OnPrerenderingFinished(child_id_, route_id_);
« no previous file with comments | « chrome/browser/prerender/prerender_contents.h ('k') | chrome/browser/prerender/prerender_final_status.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698