Fix a race condition where rapid back/forward clicks could close a tab...

chrome/browser/renderer_host/render_process_host.h

 // Copyright (c) 2009 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_RENDERER_HOST_RENDER_PROCESS_HOST_H_
 #define CHROME_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_H_
 
 #include <set>
 #include <string>
 
 #include "base/id_map.h"
 #include "base/process.h"
 #include "base/scoped_ptr.h"
 #include "chrome/common/transport_dib.h"
 #include "chrome/common/visitedlink_common.h"
 #include "ipc/ipc_sync_channel.h"
 
 class Profile;
+struct ViewMsg_ClosePage_Params;
 
 // Virtual interface that represents the browser side of the browser <->
 // renderer communication channel. There will generally be one
 // RenderProcessHost per renderer process.
 //
 // The concrete implementation of this class for normal use is the
 // BrowserRenderProcessHost. It may also be implemented by a testing interface
 // for mocking purposes.
 class RenderProcessHost : public IPC::Channel::Sender,
                           public IPC::Channel::Listener {
@@ skipping 82 matching lines @@
   virtual int GetNextRoutingID() = 0;
 
   // Called on the UI thread to cancel any outstanding resource requests for
   // the specified render widget.
   virtual void CancelResourceRequests(int render_widget_id) = 0;
 
   // Called on the UI thread to simulate a ClosePage_ACK message to the
   // ResourceDispatcherHost.  Necessary for a cross-site request, in the case
   // that the original RenderViewHost is not live and thus cannot run an
   // onunload handler.
-  virtual void CrossSiteClosePageACK(int new_render_process_host_id,
-                                     int new_request_id) = 0;
+  virtual void CrossSiteClosePageACK(
+      const ViewMsg_ClosePage_Params& params) = 0;
 
   // Called on the UI thread to wait for the next PaintRect message for the
   // specified render widget.  Returns true if successful, and the msg out-
   // param will contain a copy of the received PaintRect message.
   virtual bool WaitForPaintMsg(int render_widget_id,
                                const base::TimeDelta& max_delay,
                                IPC::Message* msg) = 0;
 
   // Called when a received message cannot be decoded.
   virtual void ReceivedBadMessage(uint16 msg_type) = 0;
@@ skipping 117 matching lines @@
 // Factory object for RenderProcessHosts. Using this factory allows tests to
 // swap out a different one to use a TestRenderProcessHost.
 class RenderProcessHostFactory {
  public:
   virtual ~RenderProcessHostFactory() {}
   virtual RenderProcessHost* CreateRenderProcessHost(
       Profile* profile) const = 0;
 };
 
 #endif  // CHROME_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_H_