For prerendering, keep track of all the intermediate redirects, and hook into...

chrome/browser/prerender/prerender_contents.h

 // 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.
 
 #ifndef CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_
 #define CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 22 matching lines @@
 // it is never allowed to navigate across a SiteInstance boundary.
 class PrerenderContents : public RenderViewHostDelegate,
                           public RenderViewHostDelegate::View,
                           public NotificationObserver,
                           public JavaScriptAppModalDialogDelegate {
  public:
   PrerenderContents(PrerenderManager* prerender_manager, Profile* profile,
                     const GURL& url);
   virtual ~PrerenderContents();
   virtual void StartPrerendering();
+  void AddAliasURLs(const std::vector<GURL>& alias_urls);

[cbentzel, 2011/01/12 23:49:53]

Why not just pass in the alias_urls to the PrerenderContents constructor? 

If you keep this method, please document it. Maybe move MatchesUrl closer.

Also, most of the other references are "matching_url" rather than "alias_url".
Be consistent with the name: I actual prefer alias, but matching is OK as well. 

[tburkard, 2011/01/13 19:14:16]

Done.

 
   RenderViewHost* render_view_host() { return render_view_host_; }
   // Allows replacing of the RenderViewHost owned by this class, including
   // replacing with a NULL value.  When a caller uses this, the caller will
   // own (and is responsible for freeing) the old RVH.
   void set_render_view_host(RenderViewHost* rvh) { render_view_host_ = rvh; }
   ViewHostMsg_FrameNavigate_Params* navigate_params() {
     return navigate_params_.get();
   }
   string16 title() const { return title_; }
   int32 page_id() const { return page_id_; }
 
+  // Indicates whether this prerendered page can be used for the provided
+  // URL, i.e. whether there is a match.
+  bool MatchesUrl(const GURL& url);

[cbentzel, 2011/01/12 23:49:53]

Nit: MatchesURL instead of MatchesUrl to be consistent.

Also, this should probably be const.

[tburkard, 2011/01/13 19:14:16]

Done.

+
   // RenderViewHostDelegate implementation.
   virtual RenderViewHostDelegate::View* GetViewDelegate();
   virtual const GURL& GetURL() const;
   virtual ViewType::Type GetRenderViewType() const;
   virtual int GetBrowserWindowID() const;
   virtual void DidNavigate(RenderViewHost* render_view_host,
                            const ViewHostMsg_FrameNavigate_Params& params);
   virtual void UpdateTitle(RenderViewHost* render_view_host,
                            int32 page_id,
                            const std::wstring& title);
@@ skipping 61 matching lines @@
                                   const std::wstring& prompt);
   virtual void SetSuppressMessageBoxes(bool suppress_message_boxes) {}
   virtual gfx::NativeWindow GetMessageBoxRootWindow();
   virtual TabContents* AsTabContents();
   virtual ExtensionHost* AsExtensionHost();
 
   virtual void UpdateInspectorSetting(const std::string& key,
                                       const std::string& value);
   virtual void ClearInspectorSettings();
 
+ protected:
+  // from renderViewHostDelegate.
+  virtual bool OnMessageReceived(const IPC::Message& message);

[cbentzel, 2011/01/12 23:49:53]

I think it's OK to make this private as well. 

[tburkard, 2011/01/13 19:14:16]

TabContents has it protected, too, so i figured protected makes more sense.

+
  private:
+  // Message handlers.
+  void OnDidStartProvisionalLoadForFrame(int64 frame_id,
+                                         bool main_frame,
+                                         const GURL& url);
+  void OnDidRedirectProvisionalLoad(int32 page_id,
+                                    const GURL& source_url,
+                                    const GURL& target_url);
+
+  void AddMatchingUrl(const GURL& url);

[cbentzel, 2011/01/12 23:49:53]

Nit: AddMatchingURL

[tburkard, 2011/01/13 19:14:16]

Done.

+
   // The prerender manager owning this object.
   PrerenderManager* prerender_manager_;
 
   // The host for our HTML content.
   RenderViewHost* render_view_host_;
 
   // Common implementations of some RenderViewHostDelegate::View methods.
   RenderViewHostDelegateViewHelper delegate_view_helper_;
 
   // The URL being prerendered.
   GURL prerender_url_;
 
   // The NavigationParameters of the finished navigation.
   scoped_ptr<ViewHostMsg_FrameNavigate_Params> navigate_params_;
 
   // The profile being used
   Profile* profile_;
 
   // Information about the title and URL of the page that this class as a
   // RenderViewHostDelegate has received from the RenderView.
   // Used to apply to the new RenderViewHost delegate that might eventually
   // own the contained RenderViewHost when the prerendered page is shown
   // in a TabContents.
   string16 title_;
   int32 page_id_;
   GURL url_;
   NotificationRegistrar registrar_;
 
+  // A vector of URLs that this prerendered page matches against.
+  // This array can contain more than element as a result of redirects,
+  // such as HTTP redirects or javascript redirects.
+  std::vector<GURL> matching_urls_;
+
   DISALLOW_COPY_AND_ASSIGN(PrerenderContents);
 };
 
 #endif  // CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_