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

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, 6 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
Index: chrome/browser/prerender/prerender_contents.cc
===================================================================
--- chrome/browser/prerender/prerender_contents.cc (revision 91002)
+++ 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, char experiment_id) OVERRIDE {
return new PrerenderContents(prerender_manager, prerender_tracker, profile,
- url, referrer, origin);
+ url, referrer, origin, experiment_id);
}
};
@@ -128,7 +128,8 @@
Profile* profile,
const GURL& url,
const GURL& referrer,
- Origin origin)
+ Origin origin,
+ char experiment_id)
: prerender_manager_(prerender_manager),
prerender_tracker_(prerender_tracker),
prerender_url_(url),
@@ -142,7 +143,8 @@
child_id_(-1),
route_id_(-1),
starting_page_id_(-1),
- origin_(origin) {
+ origin_(origin),
+ experiment_id_(experiment_id) {
DCHECK(prerender_manager != NULL);
}
@@ -313,7 +315,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_);

Powered by Google App Engine
This is Rietveld 408576698