Make browser_handles_top_level_requests synchronous.

chrome/browser/external_tab/external_tab_container_win.cc

Index: chrome/browser/external_tab/external_tab_container_win.cc
diff --git a/chrome/browser/external_tab/external_tab_container_win.cc b/chrome/browser/external_tab/external_tab_container_win.cc
index d76005913fd9cfc9842ff594cf1692403f42506f..82c2975ba506043400b763cde565d7d1746b1469 100644
--- a/chrome/browser/external_tab/external_tab_container_win.cc
+++ b/chrome/browser/external_tab/external_tab_container_win.cc
@@ -408,6 +408,23 @@ WebContents* ExternalTabContainer::OpenURLFromTab(WebContents* source,
   return NULL;
 }
 
+bool ExternalTabContainer::ShouldIgnoreNavigation(
+    WebContents* source,
+    const GURL& url,
+    const content::Referrer& referrer,
+    WindowOpenDisposition disposition,
+    bool is_content_initiated,
+    content::PageTransition transition_type) {
+  // This will make every navigation browser-initiated.
+  // TODO: Investigate modifying ExternalTabContainer to work without canceling
+  // and re-issuing every navigation.
+
+  if (is_content_initiated)
+    source->GetController().LoadURL(url, referrer, transition_type,

[ananta, 2012/04/10 19:45:45]

In ChromeFrame's case we want the top level URL request to be handled by IE,
which is achieved via the RenderViewImpl::OpenURL call in the renderer. This
code as written would just reissue the navigation in the chrome browser which is
incorrect.

[mkosiba (inactive), 2012/04/11 13:49:35]

Ok, should be better now - any ideas on how to see if my changes work?

+                                    std::string());
+  return is_content_initiated;
+}
+
 void ExternalTabContainer::NavigationStateChanged(const WebContents* source,
                                                   unsigned changed_flags) {
   if (automation_) {