PlzNavigate: implement CommitNavigation on the browser side

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
index 4342863187f19adbdbd09e942843f0f4f77eb437..4b237f1235fe8e2ee59c85e595b00c22c584767d 100644
--- a/content/browser/frame_host/navigation_request_info.h
+++ b/content/browser/frame_host/navigation_request_info.h
@@ -32,6 +32,9 @@ struct NavigationRequestInfo {
   GURL first_party_for_cookies;
   bool is_main_frame;
   bool parent_is_main_frame;
+
+  // Whether or not we should allow the URL to download.

[Charlie Reis, 2014/09/15 21:19:39]

This comment is a bit unclear.  What value does this have when it's not a
download (the common case), and how can you tell whether it's a download or not?

[clamy, 2014/09/18 20:50:39]

This is required by the loader, and I think it is true by default. I am not sure
if it can actually be false for a navigation request.

+  bool allow_download;

[Charlie Reis, 2014/09/15 21:19:39]

This is also one of the "request info" struct items.  Why isn't that struct
being used here?

 };
 
 }  // namespace content