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

Unified Diff: chrome/renderer/prerender/prerender_dispatcher.h

Issue 9416031: Prerendered pages are swapped in at browser::Navigate time. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More cleanup Created 8 years, 10 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/renderer/prerender/prerender_dispatcher.h
diff --git a/chrome/renderer/prerender/prerender_dispatcher.h b/chrome/renderer/prerender/prerender_dispatcher.h
new file mode 100644
index 0000000000000000000000000000000000000000..f6c37a2e999a55b0ce4c3937f0eb0bf32f186d96
--- /dev/null
+++ b/chrome/renderer/prerender/prerender_dispatcher.h
@@ -0,0 +1,36 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
dominich 2012/02/28 16:21:00 nit: 2012
cbentzel 2012/02/29 18:16:13 Done.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_RENDERER_PRERENDER_PRERENDER_DISPATCHER_H_
+#define CHROME_RENDERER_PRERENDER_PRERENDER_DISPATCHER_H_
+#pragma once
+
+#include <map>
+
+#include "base/compiler_specific.h"
+#include "content/public/renderer/render_process_observer.h"
+
+class GURL;
+
+namespace prerender {
+
+// PrerenderDispatcher pays attention to which URLs are prerendered.
dominich 2012/02/28 16:21:00 nit: 'pays attention to' is a bit vague. What does
cbentzel 2012/02/29 18:16:13 Improved description.
+class PrerenderDispatcher : public content::RenderProcessObserver {
+ public:
+ bool IsPrerenderURL(const GURL & url) const;
+
+ private:
+ void OnAddPrerenderURL(const GURL& url);
+ void OnRemovePrerenderURL(const GURL& url);
+
+ // RenderProcessObserver:
+ virtual bool OnControlMessageReceived(const IPC::Message& message) OVERRIDE;
+
+ typedef std::map<GURL, int> PrerenderMap;
+ std::map<GURL, int> prerender_urls_;
dominich 2012/02/28 16:21:00 nit: use PrerenderMap in this declaration.
cbentzel 2012/02/29 18:16:13 Nice catch. Done.
+};
+
+} // namespace prerender
+
+#endif // CHROME_RENDERER_PRERENDER_PRERENDER_HELPER_H_

Powered by Google App Engine
This is Rietveld 408576698