Add a FrameHostMsg_BeginNavigation IPC

content/browser/frame_host/navigation_request_info.h

Index: content/browser/frame_host/navigation_request_info.h
diff --git a/content/browser/frame_host/navigation_request_info.h b/content/browser/frame_host/navigation_request_info.h
new file mode 100644
index 0000000000000000000000000000000000000000..c83b716c2976214bc82e52b70952fb982e78f9b8
--- /dev/null
+++ b/content/browser/frame_host/navigation_request_info.h
@@ -0,0 +1,71 @@
+// Copyright 2014 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_FRAME_HOST_NAVIGATION_REQUEST_INFO_H_
+#define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_REQUEST_INFO_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "content/public/common/page_transition_types.h"
+#include "content/public/common/referrer.h"
+#include "url/gurl.h"
+
+struct FrameHostMsg_BeginNavigation_Params;
+
+namespace content {
+
+// A struct to hold the parameters needed to start a navigation request in
+// ResourceDispatcherHost. It is initialized on the UI thread, and then passed
+// to the IO thread by a NavigationRequest object.
+struct NavigationRequestInfo {
+  NavigationRequestInfo(const FrameHostMsg_BeginNavigation_Params& params);
+
+  // The following parameters should be initialized using a
+  // FrameHostMsg_BeginNavigation_Params.

[jam, 2014/07/15 00:05:43]

did you consider having a FrameHostMsg_BeginNavigation_Params as a member here
since it's almost the same as these first 9 parameters? is the presence of two
unused parameters in that struct important?

[clamy, 2014/07/15 15:32:14]

Done.

+
+  // The request method: GET, POST, etc.
+  std::string method;
+
+  // The requested URL.
+  GURL url;
+
+  // The referrer to use.
+  Referrer referrer;
+
+  // Additional HTTP request headers.
+  std::string headers;
+
+  // net::URLRequest load flags (net::LOAD_NORMAL | net::LOAD_ENABLE_LOAD_TIMING
+  // by default).
+  int load_flags;
+
+  // True if the request was user initiated.
+  bool has_user_gesture;
+  PageTransition transition_type;
+
+  // Whether this navigation should replace the current session history entry on
+  // commit.
+  bool should_replace_current_entry;
+
+  // Whether or not we should allow the URL to download.
+  bool allow_download;
+
+  // ---------------------------------------------------------------------------
+  // The following parameters should be filled in by RenderFrameHostManager
+  // before the navigation request is sent to the ResourceDispatcherHost.
+
+  // Usually the URL of the document in the top-level window, which may be
+  // checked by the third-party cookie blocking policy.
+  GURL first_party_for_cookies;
+  bool is_main_frame;
+  bool parent_is_main_frame;
+
+  // True if the frame is visible.
+  bool is_showing;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_REQUEST_INFO_H_