Add Perceived PageLoad Time (PPLT) metrics for Prerender Experiments,...

chrome/browser/tab_contents/tab_contents.cc

Index: chrome/browser/tab_contents/tab_contents.cc
===================================================================
--- chrome/browser/tab_contents/tab_contents.cc	(revision 72357)
+++ chrome/browser/tab_contents/tab_contents.cc	(working copy)
@@ -52,6 +52,7 @@
 #include "chrome/browser/plugin_installer_infobar_delegate.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prerender/prerender_manager.h"
+#include "chrome/browser/prerender/prerender_plt_recorder.h"
 #include "chrome/browser/printing/print_preview_tab_controller.h"
 #include "chrome/browser/printing/print_view_manager.h"
 #include "chrome/browser/profiles/profile.h"
@@ -367,6 +368,7 @@
       autocomplete_history_manager_(),
       autofill_manager_(),
       plugin_installer_(),
+      prerender_plt_recorder_(),
       bookmark_drag_(NULL),
       ALLOW_THIS_IN_INITIALIZER_LIST(fav_icon_helper_(this)),
       is_loading_(false),
@@ -472,6 +474,8 @@
   AddNavigationObserver(autofill_manager_.get());
   autocomplete_history_manager_.reset(new AutocompleteHistoryManager(this));
   AddNavigationObserver(autocomplete_history_manager_.get());
+  prerender_plt_recorder_.reset(new PrerenderPLTRecorder(this));
+  AddNavigationObserver(prerender_plt_recorder_.get());

[brettw, 2011/01/26 22:13:28]

Hmm, I was hoping everything could be in your component, and now I see what you
mean by it being more work by needing an extra class. But it looks like how I
was imagining this would work isn't entirely supported by the current design. So
this is fine.

 }
 
 TabContents::~TabContents() {
@@ -3393,12 +3397,8 @@
 bool TabContents::MaybeUsePreloadedPage(const GURL& url) {
   PrerenderManager* pm = profile()->GetPrerenderManager();
   if (pm != NULL) {
-    if (pm->MaybeUsePreloadedPage(this, url)) {
-      // TODO(tburkard): If the preloaded page has not finished preloading
-      // yet, we should not do this.
-      DidStopLoading();
+    if (pm->MaybeUsePreloadedPage(this, url))
       return true;
-    }
   }
   return false;
 }