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

chrome/browser/prerender/prerender_contents.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/prerender/prerender_contents.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/background_contents_service.h"
 #include "chrome/browser/browsing_instance.h"
 #include "chrome/browser/prerender/prerender_manager.h"
@@ skipping 10 matching lines @@
 #include "gfx/rect.h"
 
 PrerenderContents::PrerenderContents(PrerenderManager* prerender_manager,
                                      Profile* profile,
                                      const GURL& url,
                                      const std::vector<GURL>& alias_urls)
     : prerender_manager_(prerender_manager),
       render_view_host_(NULL),
       prerender_url_(url),
       profile_(profile),
-      page_id_(0) {
+      page_id_(0),
+      has_stopped_loading_(false) {
   DCHECK(prerender_manager != NULL);
   AddAliasURL(prerender_url_);
   for (std::vector<GURL>::const_iterator it = alias_urls.begin();
        it != alias_urls.end();
        ++it) {
     AddAliasURL(*it);
   }
 }
 
 void PrerenderContents::StartPrerendering() {
@@ skipping 74 matching lines @@
 }
 
 void PrerenderContents::RunJavaScriptMessage(
     const std::wstring& message,
     const std::wstring& default_prompt,
     const GURL& frame_url,
     const int flags,
     IPC::Message* reply_msg,
     bool* did_suppress_message) {
   *did_suppress_message = true;
+  // Cancel prerendering, since we do cannot deal with a pop up message.
+  prerender_manager_->RemoveEntry(this);

[cbentzel, 2011/01/26 19:38:36]

Move this to a different CL.

[tburkard, 2011/01/26 20:05:50]

Done.

 }
 
 bool PrerenderContents::PreHandleKeyboardEvent(
     const NativeWebKeyboardEvent& event,
     bool* is_keyboard_shortcut) {
   return false;
 }
 
 void PrerenderContents::Observe(NotificationType type,
                                 const NotificationSource& source,
@@ skipping 122 matching lines @@
 }
 
 void PrerenderContents::AddAliasURL(const GURL& url) {
   alias_urls_.push_back(url);
 }
 
 bool PrerenderContents::MatchesURL(const GURL& url) const {
   return std::find(alias_urls_.begin(), alias_urls_.end(), url)
       != alias_urls_.end();
 }
+
+void PrerenderContents::DidStopLoading() {

[cbentzel, 2011/01/26 19:38:36]

DCHECK(has_stopped_loading_) - can this ever be called multiple times in client
issued redirect case? 

[tburkard, 2011/01/26 20:05:50]

Could happen multiple times.
In some pages, the loading restarts sometimes for some reason.

On 2011/01/26 19:38:36, cbentzel wrote:

+  has_stopped_loading_ = true;
+}