To fix the cross-site post submission bug.

content/public/browser/page_navigator.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.
 
 // PageNavigator defines an interface that can be used to express the user's
 // intention to navigate to a particular URL.  The implementing class should
 // perform the navigation.
 
 #ifndef CONTENT_PUBLIC_BROWSER_PAGE_NAVIGATOR_H_
 #define CONTENT_PUBLIC_BROWSER_PAGE_NAVIGATOR_H_
 
 #include <string>
 
 #include "content/common/content_export.h"
 #include "content/public/browser/global_request_id.h"
+#include "content/public/common/frame_navigate_params.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/common/referrer.h"
 #include "googleurl/src/gurl.h"
 #include "webkit/glue/window_open_disposition.h"
 
 namespace content {
 
 class WebContents;
+struct ViewMsg_Request;
 
 struct CONTENT_EXPORT OpenURLParams {
   OpenURLParams(const GURL& url,
                 const Referrer& referrer,
                 WindowOpenDisposition disposition,
                 PageTransition transition,
                 bool is_renderer_initiated);
   OpenURLParams(const GURL& url,
                 const Referrer& referrer,
                 int64 source_frame_id,
@@ skipping 23 matching lines @@
   // Whether this navigation is initiated by the renderer process.
   bool is_renderer_initiated;
 
   // The override encoding of the URL contents to be opened.
   std::string override_encoding;
 
   // Reference to the old request id in case this is a navigation that is being
   // transferred to a new renderer.
   GlobalRequestID transferred_global_request_id;
 
+  // The Post data in HTTPRequest body.
+  ViewMsg_Request request;

[michaeln, 2012/11/02 00:08:23]

There's some overlap between member of tha OpenURLParams struct and members of
the new 'request' struct, in particular the 'url' members. The new info that's
missing from this struct seems to be the post data and the content type. I
wonder if the content type can go in the extra_headers field? The new transition
type implies 'POST' so not really sure we need the method string?

[irobert, 2012/11/02 17:22:55]

Besides the url member, the other member in the ViewMsg_Request does not fully
support on all the param structs which needs to be used to pass the request.

On 2012/11/02 00:08:23, michaeln wrote:

+
  private:
   OpenURLParams();
 };
 
 class PageNavigator {
  public:
   virtual ~PageNavigator() {}
 
   // Opens a URL with the given disposition.  The transition specifies how this
   // navigation should be recorded in the history system (for example, typed).
   // Returns the WebContents the URL is opened in, or NULL if the URL wasn't
   // opened immediately.
   virtual WebContents* OpenURL(const OpenURLParams& params) = 0;
 };
 
 }
 
 #endif  // CONTENT_PUBLIC_BROWSER_PAGE_NAVIGATOR_H_