PlzNavigate: implement CommitNavigation on the browser side

content/common/frame_messages.h

Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index d77d9e6328d2ec9e3b4200f2a7266d1cbd211028..b13942868314767073cda73c060802c624e2dda1 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -296,13 +296,62 @@ IPC_STRUCT_BEGIN(FrameHostMsg_BeginNavigation_Params)
   IPC_STRUCT_MEMBER(bool, has_user_gesture)
 
   IPC_STRUCT_MEMBER(content::PageTransition, transition_type)
+IPC_STRUCT_END()
 
-  // Whether this navigation should replace the current session history entry on
-  // commit.
-  IPC_STRUCT_MEMBER(bool, should_replace_current_entry)
+IPC_STRUCT_BEGIN(FrameMsg_CommitNavigation_Params)
+  // The URL to commit.
+  IPC_STRUCT_MEMBER(GURL, url)

[(Do not use) nasko, 2014/09/05 17:04:49]

Yeah, this is the point I was trying to make. This message has a huge overlap
with FrameMsg_Navigate_Params. We can always pass two structs in an IPC:

Send(new FrameMsg_Navigate(FrameMsg_NavigationParams, FrameMsg_Commit_Params))

This way we can "compose" all parameters based on the common and specific ones.

[clamy, 2014/09/05 18:21:25]

So should this be split up in something like FrameMsg_CoreNavigation_Params,
FrameMsg_CommitNavigation_Params and FrameMsg_Navigate_Params? Or maybe define a
content::NavigationCoreParams and have both FrameMsg_CommitNavigation_Params and
FrameMsg_Navigate_Params be IPC struct with parent as is done for
FrameHostMsg_DidCommitProvisionalLoad_Params?

 
-  // Whether or not we should allow the URL to download.
-  IPC_STRUCT_MEMBER(bool, allow_download)
+  // The URL to request to the browser to get access to the stream of data.
+  IPC_STRUCT_MEMBER(GURL, stream_url)
+
+  // The page_id for this navigation, or -1 if it is a new navigation.  Back,
+  // Forward, and Reload navigations should have a valid page_id.  If the load
+  // succeeds, then this page_id will be reflected in the resultant
+  // FrameHostMsg_DidCommitProvisionalLoad message.
+  IPC_STRUCT_MEMBER(int32, page_id)
+
+  // If page_id is -1, then pending_history_list_offset will also be -1.
+  // Otherwise, it contains the offset into the history list corresponding to
+  // the current navigation.
+  IPC_STRUCT_MEMBER(int, pending_history_list_offset)
+
+  // Informs the RenderView of where its current page contents reside in
+  // session history and the total size of the session history list.
+  IPC_STRUCT_MEMBER(int, current_history_list_offset)
+  IPC_STRUCT_MEMBER(int, current_history_list_length)
+
+  // Informs the RenderView the session history should be cleared. In that
+  // case, the RenderView needs to notify the browser that the clearing was
+  // succesful when the navigation commits.
+  IPC_STRUCT_MEMBER(bool, should_clear_history_list)
+
+  // TODO(clamy): Investigate whether we should support data urls here.
+
+  // The URL to send in the "Referer" header field. Can be empty if there is
+  // no referrer.
+  IPC_STRUCT_MEMBER(content::Referrer, referrer)
+
+  // TODO(clamy): Investigate whether we should send the chain of redirects
+  // along.
+
+  // The type of transition.
+  IPC_STRUCT_MEMBER(content::PageTransition, transition)
+
+  // Opaque history state (received by ViewHostMsg_UpdateState).
+  IPC_STRUCT_MEMBER(content::PageState, page_state)
+
+  // Type of navigation.
+  IPC_STRUCT_MEMBER(FrameMsg_Navigate_Type::Value, navigation_type)
+
+  // Whether or not the user agent override string should be used.
+  IPC_STRUCT_MEMBER(bool, is_overriding_user_agent)
+
+  // The navigationStart time to expose through the Navigation Timing API to JS.
+  IPC_STRUCT_MEMBER(base::TimeTicks, browser_navigation_start)
+
+  // TODO(clamy): Add other timing values needed by the Navigation Timing API
+  // (redirects, etc...).
 IPC_STRUCT_END()
 
 #if defined(OS_MACOSX) || defined(OS_ANDROID)
@@ -481,6 +530,10 @@ IPC_MESSAGE_ROUTED1(FrameMsg_SelectPopupMenuItem,
 
 #endif
 
+// Tells the renderer to commit the specific navigation, as part of the
+// browser-side navigation refactoring.
+IPC_MESSAGE_ROUTED1(FrameMsg_CommitNavigation, FrameMsg_CommitNavigation_Params)
+
 // -----------------------------------------------------------------------------
 // Messages sent from the renderer to the browser.