New link rel=prerender api, using WebKit::WebPrerenderingPlatform

chrome/browser/prerender/prerender_message_filter.h

Index: chrome/browser/prerender/prerender_message_filter.h
diff --git a/chrome/browser/prerender/prerender_message_filter.h b/chrome/browser/prerender/prerender_message_filter.h
new file mode 100644
index 0000000000000000000000000000000000000000..3c8dfba41b702e60ecca256179044eedb6a87b94
--- /dev/null
+++ b/chrome/browser/prerender/prerender_message_filter.h
@@ -0,0 +1,59 @@
+// Copyright (c) 2012 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_MESSAGE_FILTER_H_
+#define CHROME_BROWSER_PRERENDER_PRERENDER_MESSAGE_FILTER_H_
+
+#include "base/compiler_specific.h"
+#include "content/public/browser/browser_message_filter.h"
+#include "googleurl/src/gurl.h"
+
+class Profile;
+
+namespace content {
+struct Referrer;
+}
+
+namespace gfx {
+class Size;
+}
+
+namespace IPC {
+class Message;
+}
+
+namespace prerender {
+
+class PrerenderMessageFilter : public content::BrowserMessageFilter {
+ public:
+  PrerenderMessageFilter(int render_process_id, Profile* profile);
+
+  virtual bool OnMessageReceived(const IPC::Message& message,
+                                 bool* message_was_ok) OVERRIDE;
+  virtual void OverrideThreadForMessage(
+      const IPC::Message& message,
+      content::BrowserThread::ID* thread) OVERRIDE;

[mmenke, 2012/05/01 16:23:21]

Any reason why these functions are public?

[gavinp, 2012/05/01 18:50:22]

None, so I fixed it.  The convention is for them to be public, though.

[gavinp, 2012/05/01 18:50:22]

No, none.  Fixed.

The convention in Chrome is for them to be public, but that's clearly not
required.

[mmenke, 2012/05/02 15:02:52]

Indeed.  I wouldn't have commented, if OnChannelClosing wasn't private.  I'm
happy either way, as lon as they're together.

+
+ private:
+  virtual ~PrerenderMessageFilter();
+
+  void OnAddPrerender(int prerender_id,
+                      const GURL& url,
+                      const content::Referrer& referrer,
+                      const gfx::Size& size,
+                      int render_view_route_id);
+  void OnCancelPrerender(int prerender_id);
+  void OnAbandonPrerender(int prerender_id);
+  virtual void OnChannelClosing() OVERRIDE;

[mmenke, 2012/05/01 16:23:21]

nit:  Should have a comment that the OVERRIDE functions are implementing
content::BrowserMessageFilter.

[gavinp, 2012/05/01 18:50:22]

Done.

+
+  const int render_process_id_;
+  Profile* const profile_;
+
+  DISALLOW_COPY_AND_ASSIGN(PrerenderMessageFilter);
+};
+
+}  // namespace prerender
+
+#endif  // CHROME_BROWSER_PRERENDER_PRERENDER_MESSAGE_FILTER_H_
+