Handle interface to prerenders.

chrome/browser/prerender/prerender_link_manager.h

Index: chrome/browser/prerender/prerender_link_manager.h
diff --git a/chrome/browser/prerender/prerender_link_manager.h b/chrome/browser/prerender/prerender_link_manager.h
index aacee8bc4cdb08b906d4190ff9f760065afba0ef..7e9a5e4251af296086461a23a86a0e519c8c095b 100644
--- a/chrome/browser/prerender/prerender_link_manager.h
+++ b/chrome/browser/prerender/prerender_link_manager.h
@@ -26,6 +26,7 @@ class Size;
 
 namespace prerender {
 
+class PrerenderHandle;
 class PrerenderManager;
 
 // PrerenderLinkManager implements the API on Link elements for all documents
@@ -70,14 +71,19 @@ class PrerenderLinkManager : public ProfileKeyedService {
   friend class PrerenderTest;
 
   typedef std::pair<int, int> ChildAndPrerenderIdPair;
-  typedef std::map<ChildAndPrerenderIdPair, GURL> IdPairToUrlMap;
+  typedef std::map<ChildAndPrerenderIdPair, PrerenderHandle*>
+      IdPairToPrerenderHandleMap;
 
-  void RemovePrerender(const IdPairToUrlMap::iterator& id_url_iter);
+  void RemovePrerender(const IdPairToPrerenderHandleMap::iterator& id_url_iter);

[gavinp, 2012/07/11 17:04:00]

Apparently, I was thinking about this function too.

 
   bool IsEmpty() const;
 
   PrerenderManager* manager_;
-  IdPairToUrlMap ids_to_url_map_;
+
+  // Map giving, per child process, for each prerender_id, the PrerenderHandle.

[mmenke, 2012/07/09 18:06:57]

nit:  prerender id (prerender_id is not a type, but rather an argument name to
some methods).

[mmenke, 2012/07/09 18:06:57]

nit:  I find this sentence rather hard to follow.  I'd suggest "A map from child
process id and prerender id to PrerenderHandles.  Prerender ids are only unique
within each renderer process...."

[gavinp, 2012/07/11 17:04:00]

Done.

+  // This map owns the PrerenderHandle, and so when erasing, you should delete
+  // the handle too.
+  IdPairToPrerenderHandleMap ids_to_handle_map_;
 
   DISALLOW_COPY_AND_ASSIGN(PrerenderLinkManager);
 };