Make browser_handles_top_level_requests synchronous.

content/browser/tab_contents/tab_contents.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/tab_contents/tab_contents.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
@@ skipping 2294 matching lines @@
     FOR_EACH_OBSERVER(WebContentsObserver, observers_,
                       DidOpenRequestedURL(new_contents,
                                           url,
                                           referrer,
                                           disposition,
                                           transition_type,
                                           source_frame_id));
   }
 }
 
+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,
     const string16& default_prompt,
     const GURL& frame_url,
     ui::JavascriptMessageType javascript_message_type,
     IPC::Message* reply_msg,
     bool* did_suppress_message) {
   // Suppress JavaScript dialogs when requested. Also suppress messages when
   // showing an interstitial as it's shown over the previous page and we don't
@@ skipping 271 matching lines @@
 void TabContents::CreateViewAndSetSizeForRVH(RenderViewHost* rvh) {
   RenderWidgetHostView* rwh_view = GetView()->CreateViewForWidget(rvh);
   // Can be NULL during tests.
   if (rwh_view)
     rwh_view->SetSize(GetView()->GetContainerSize());
 }
 
 RenderViewHostImpl* TabContents::GetRenderViewHostImpl() {
   return static_cast<RenderViewHostImpl*>(GetRenderViewHost());
 }