Make browser_handles_top_level_requests synchronous.

content/browser/renderer_host/render_view_host_impl.cc

Index: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index e31de0f182530c5e9c400978464681233c61a479..9c6080368ae92768758eba5278d8ac1c1961a886 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -828,6 +828,8 @@ bool RenderViewHostImpl::OnMessageReceived(const IPC::Message& msg) {
     IPC_MESSAGE_HANDLER(ViewHostMsg_ToggleFullscreen,
                         OnMsgToggleFullscreen)
     IPC_MESSAGE_HANDLER(ViewHostMsg_OpenURL, OnMsgOpenURL)
+    IPC_MESSAGE_HANDLER(ViewHostMsg_ShouldIgnoreNavigation,
+                        OnMsgShouldIgnoreNavigation)
     IPC_MESSAGE_HANDLER(ViewHostMsg_DidContentsPreferredSizeChange,
                         OnMsgDidContentsPreferredSizeChange)
     IPC_MESSAGE_HANDLER(ViewHostMsg_DidChangeScrollbarsForMainFrame,
@@ -1160,6 +1162,21 @@ void RenderViewHostImpl::OnMsgOpenURL(const GURL& url,
       validated_url, referrer, disposition, source_frame_id);
 }
 
+void RenderViewHostImpl::OnMsgShouldIgnoreNavigation(
+    const GURL& url,
+    const content::Referrer& referrer,
+    WindowOpenDisposition disposition,
+    int64 source_frame_id,
+    bool is_content_initiated,
+    bool* result) {
+  GURL validated_url(url);
+  FilterURL(ChildProcessSecurityPolicyImpl::GetInstance(),
+            GetProcess()->GetID(), false, &validated_url);
+  *result = delegate_->ShouldIgnoreNavigation(validated_url, referrer,
+                                              disposition, source_frame_id,

[groby-ooo-7-16, 2012/04/05 22:51:19]

nit: collapse last two argument lines into one. Save a line :)

[mkosiba (inactive), 2012/04/10 17:58:37]

Would this be OK according to the code formatting guide?

  *result = delegate_->ShouldIgnoreNavigation(validated_url, referrer,
      disposition, source_frame_id, is_content_initiated);

I thought the only alternative is:

  *result = delegate_->ShouldIgnoreNavigation(
      validated_url, referrer, disposition, source_frame_id,
      is_content_initiated);

+                                              is_content_initiated);
+}
+
 void RenderViewHostImpl::OnMsgDidContentsPreferredSizeChange(
     const gfx::Size& new_size) {
   delegate_->UpdatePreferredSize(new_size);