To fix the cross-site post submission bug.

content/browser/renderer_host/render_view_host_delegate.h

 // 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 CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/i18n/rtl.h"
 #include "base/process_util.h"
 #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"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebPopupType.h"
 #include "webkit/glue/window_open_disposition.h"
 
 class GURL;
 class SkBitmap;
 class WebKeyboardEvent;
 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;
 }
 
 namespace base {
 class ListValue;
 class TimeTicks;
 }
 
@@ skipping 18 matching lines @@
 class RenderViewHostDelegateView;
 class SessionStorageNamespace;
 class WebContents;
 class WebContentsImpl;
 struct ContextMenuParams;
 struct FileChooserParams;
 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;
 
 //
 // RenderViewHostDelegate
 //
 //  An interface implemented by an object interested in knowing about the state
 //  of the RenderViewHost.
 //
 //  This interface currently encompasses every type of message that was
@@ skipping 336 matching lines @@
   // calling |callback|.
   virtual void RequestMediaAccessPermission(
       const MediaStreamRequest* request,
       const MediaResponseCallback& callback) {}
 
 #if defined(OS_ANDROID)
   virtual void AttachLayer(WebKit::WebLayer* layer) {}
   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() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_