To fix the cross-site post submission bug.

content/browser/renderer_host/render_view_host_delegate.h

Index: content/browser/renderer_host/render_view_host_delegate.h
diff --git a/content/browser/renderer_host/render_view_host_delegate.h b/content/browser/renderer_host/render_view_host_delegate.h
index 784ea53af0004fe126cabe7e531f323c8d4eb296..6120ffbc5d519fd52f5816347e2e1ae7b786d31b 100644
--- a/content/browser/renderer_host/render_view_host_delegate.h
+++ b/content/browser/renderer_host/render_view_host_delegate.h
@@ -14,6 +14,7 @@
 #include "base/string16.h"
 #include "content/common/content_export.h"
 #include "content/public/common/context_menu_source_type.h"
+#include "content/public/common/frame_navigate_params.h"
 #include "content/public/common/javascript_message_type.h"
 #include "content/public/common/media_stream_request.h"
 #include "net/base/load_states.h"
@@ -27,6 +28,7 @@ struct ViewHostMsg_CreateWindow_Params;
 struct ViewHostMsg_DidFailProvisionalLoadWithError_Params;
 struct ViewHostMsg_FrameNavigate_Params;
 struct ViewMsg_PostMessage_Params;
+struct ViewMsg_PostRequest_Params;
 
 namespace webkit_glue {
 struct WebPreferences;
@@ -65,6 +67,7 @@ struct GlobalRequestID;
 struct NativeWebKeyboardEvent;
 struct Referrer;
 struct RendererPreferences;
+struct ViewMsg_Request;

[michaeln, 2012/11/05 23:38:29]

stray code from before

[irobert, 2012/11/06 05:39:20]

Done.

 
 typedef base::Callback< void(const MediaStreamDevices&) > MediaResponseCallback;
 
@@ -421,6 +424,24 @@ class CONTENT_EXPORT RenderViewHostDelegate {
   virtual void RemoveLayer(WebKit::WebLayer* layer) {}
 #endif
 
+
+  // For Cross-Navigation Post Submisstion
+  virtual void RequestOpenPostURL(

[michaeln, 2012/11/05 23:38:29]

In previous snapshots you had it as an open question whether the exising
RequestOpen interfaces should be widened to accept optional additional POST data
params or new parallel interfaces for the POST case would be preferable.

In looking at the parallel method bodies, there's some of duplicated code in
there that probably can be eliminated. I think I'd lean towards merging OpenURL
and OpenPostURL since most of the logic is the same (modulo passing the POST
data around).

[irobert, 2012/11/06 05:39:20]

Yes. I am also have the same feeling, since the only purpose of OpenPostURL is
to pass the extra POST data struct to browser process.

On 2012/11/05 23:38:29, michaeln wrote:

+      RenderViewHost* rvh,
+      const GURL& url,
+      const Referrer& referrer,
+      WindowOpenDisposition disposition,
+      int64 source_frame_id,
+      const ViewMsg_PostRequest_Params& request) {}
+
+  virtual void RequestTransferPostURL(
+      const GURL& url,
+      const Referrer& referrer,
+      WindowOpenDisposition disposition,
+      int64 source_frame_id,
+      const GlobalRequestID& old_request_id,
+      const ViewMsg_PostRequest_Params& request) {}
+
  protected:
   virtual ~RenderViewHostDelegate() {}
 };