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

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

Issue 11028037: Fix prerender histograms for multiple prerender case. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update the rite cl Created 8 years, 2 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_tab_helper.cc
diff --git a/chrome/browser/prerender/prerender_tab_helper.cc b/chrome/browser/prerender/prerender_tab_helper.cc
index 5d4e011875093c4216b7ef5756798711a8f19157..e10b034627ddbdae209a5dddd4a090da12ffc0e2 100644
--- a/chrome/browser/prerender/prerender_tab_helper.cc
+++ b/chrome/browser/prerender/prerender_tab_helper.cc
@@ -90,8 +90,8 @@ class PrerenderTabHelper::PixelStats {
PrerenderManager* prerender_manager =
tab_helper_->MaybeGetPrerenderManager();
if (prerender_manager) {
- prerender_manager->histograms()->RecordFractionPixelsFinalAtSwapin(
- CompareBitmaps(bitmap_.get(), bitmap.get()));
+ prerender_manager->RecordFractionPixelsFinalAtSwapin(
+ web_contents, CompareBitmaps(bitmap_.get(), bitmap.get()));
}
bitmap_.reset();
bitmap_web_contents_ = NULL;
@@ -228,7 +228,7 @@ bool PrerenderTabHelper::IsPrerendered() {
PrerenderManager* prerender_manager = MaybeGetPrerenderManager();
if (!prerender_manager)
return false;
- return prerender_manager->IsWebContentsPrerendered(web_contents());
+ return prerender_manager->IsWebContentsPrerendered(web_contents(), NULL);
}
void PrerenderTabHelper::PrerenderSwappedIn() {
@@ -240,7 +240,7 @@ void PrerenderTabHelper::PrerenderSwappedIn() {
web_contents(), url_);
PrerenderManager* prerender_manager = MaybeGetPrerenderManager();
if (prerender_manager)
- prerender_manager->histograms()->RecordFractionPixelsFinalAtSwapin(1.0);
+ prerender_manager->RecordFractionPixelsFinalAtSwapin(web_contents(), 1.0);
} else {
// If we have not finished loading yet, record the actual load start, and
// rebase the start time to now.
« chrome/browser/prerender/prerender_origin.h ('K') | « chrome/browser/prerender/prerender_origin.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698