To fix the cross-site post submission bug.

chrome/browser/ui/browser_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.
 
 #ifndef CHROME_BROWSER_UI_BROWSER_NAVIGATOR_H_
 #define CHROME_BROWSER_UI_BROWSER_NAVIGATOR_H_
 
 #include <string>
 
+#include "content/public/common/frame_navigate_params.h"
+
+#include "base/memory/ref_counted.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/global_request_id.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/common/referrer.h"
 #include "googleurl/src/gurl.h"
 #include "ui/gfx/rect.h"
 #include "webkit/glue/window_open_disposition.h"
 
 class Browser;
 class Profile;
 class TabContents;
+using content::WebHTTPPOSTBodyParams;
 
 namespace chrome {
 
 // Parameters that tell Navigate() what to do.
 //
 // Some basic examples:
 //
 // Simple Navigate to URL in current tab:
 // chrome::NavigateParams params(browser, GURL("http://www.google.com/"),
 //                               content::PAGE_TRANSITION_LINK);
@@ skipping 164 matching lines @@
 
   // Refers to a navigation that was parked in the browser in order to be
   // transferred to another RVH. Only used in case of a redirection of a request
   // to a different site that created a new RVH.
   content::GlobalRequestID transferred_global_request_id;
 
   // Refers to which desktop this navigation should occur on. May be passed
   // explicitly or inferred from an existing Browser instance.
   chrome::HostDesktopType host_desktop_type;
 
+  // post data
+  std::vector<content::WebHTTPPOSTBodyParams> post_data;

[michaeln, 2012/10/23 23:22:18]

How is this construct different than ResourceRequestBody::Element?

[irobert, 2012/10/24 00:51:17]

This is the exactly same as the ResourceRequestBody::Element. My purpose of
doing this is to pass all the elements in the post request to the new process
through IPC. Since you cannot pass an instance of a class through IPC (based on
my current understanding of the code.), I create this struct and convert each
element to this.

On 2012/10/23 23:22:18, michaeln wrote:

[michaeln, 2012/10/24 23:45:39]

Looks like ResourceRequestBody and it's Element data type do have ipc message
traits so they can be passed around in messages. If possible, it'd be better to
reuse that type instead of creating a new type here.

+
  private:
   NavigateParams();
 };
 
 // Navigates according to the configuration specified in |params|.
 void Navigate(NavigateParams* params);
 
 // Returns true if the url is allowed to open in incognito window.
 bool IsURLAllowedInIncognito(const GURL& url,
                              content::BrowserContext* browser_context);
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_UI_BROWSER_NAVIGATOR_H_