Make browser_handles_top_level_requests synchronous.

content/browser/tab_contents/tab_contents.cc

Index: content/browser/tab_contents/tab_contents.cc
diff --git a/content/browser/tab_contents/tab_contents.cc b/content/browser/tab_contents/tab_contents.cc
index cd366410897d73ee92f39b24068f817cf47efdfc..bf59912ccefa56e5bb156a1da218932e055bcd47 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -2312,6 +2312,24 @@ void TabContents::RequestTransferURL(const GURL& url,
   }
 }
 
+bool TabContents::ShouldIgnoreNavigation(
+    const GURL& url,
+    const content::Referrer& referrer,
+    WindowOpenDisposition disposition,
+    int64 source_frame_id,
+    bool is_content_initiated) {
+  if (!delegate_)
+    return false;
+  content::PageTransition transition_type = content::PAGE_TRANSITION_LINK;
+  NavigationEntry* active_navigation_entry = controller_.GetActiveEntry();
+  if (active_navigation_entry) {
+    transition_type = active_navigation_entry->GetTransitionType();
+  }
+  return delegate_->ShouldIgnoreNavigation(this, url, referrer, disposition,
+                                           is_content_initiated,
+                                           transition_type);

[Charlie Reis, 2012/04/05 17:06:53]

I'm not very familiar with ExternalTabContainers.  In which situations is
delegate_ one?

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

Neither am I. I don't even have a Windows machine to test this out on.
This is what I found on one of the mailing list:


ExternalTabContainer is an abstraction that replaces Browser in the case 
where the host is not a Chrome window.  Chrome Frame uses this to host 
Chrome-rendered content in a plugin HWND, generally inside IE. 

Form searching the code I see it being used in automation_provider_win.cc. I'm
adding ananta@ who seems to have authored a lot of the code in
ExternalTabContainer.

+}
+
 void TabContents::RunJavaScriptMessage(
     RenderViewHost* rvh,
     const string16& message,