Make browser_handles_top_level_requests synchronous.

content/renderer/render_view_impl.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/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 2307 matching lines @@
     // TODO(cevans): revisit whether this origin check is still necessary once
     // crbug.com/101395 is fixed.
     if (frame_url.GetOrigin() != url.GetOrigin()) {
       OpenURL(frame, url, referrer, default_policy);
       return WebKit::WebNavigationPolicyIgnore;
     }
   }
 
   // If the browser is interested, then give it a chance to look at top level
   // navigations.
-  if (is_content_initiated) {
-    bool browser_handles_top_level_requests =
-        renderer_preferences_.browser_handles_top_level_requests &&
-        IsNonLocalTopLevelNavigation(url, frame, type);
-    if (browser_handles_top_level_requests ||
+  if (renderer_preferences_.browser_handles_top_level_requests) {
+    if (IsNonLocalTopLevelNavigation(url, frame, type) ||
         renderer_preferences_.browser_handles_all_requests) {
       // Reset these counters as the RenderView could be reused for the next
       // navigation.
       page_id_ = -1;
       last_page_id_sent_to_browser_ = -1;
-      OpenURL(frame, url, referrer, default_policy);
-      return WebKit::WebNavigationPolicyIgnore;  // Suppress the load here.
+      bool ignore_navigation = false;
+      Send(new ViewHostMsg_ShouldIgnoreNavigation(
+          routing_id_,
+          url,
+          referrer,
+          NavigationPolicyToDisposition(default_policy),
+          frame->identifier(),
+          is_content_initiated,
+          &ignore_navigation));

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

Would not do the right thing for ChromeFrame as mentioned in the comment in
ExternalTabContainer.

+      if (ignore_navigation)
+        return WebKit::WebNavigationPolicyIgnore;
     }
   }
 
   // Detect when we're crossing a permission-based boundary (e.g. into or out of
   // an extension or app origin, leaving a WebUI page, etc). We only care about
   // top-level navigations (not iframes). But we sometimes navigate to
   // about:blank to clear a tab, and we want to still allow that.
   //
   // Note: this is known to break POST submissions when crossing process
   // boundaries until http://crbug.com/101395 is fixed.  This is better for
@@ skipping 2883 matching lines @@
 bool RenderViewImpl::WebWidgetHandlesCompositorScheduling() const {
   return !!RenderThreadImpl::current()->compositor_thread();
 }
 
 void RenderViewImpl::OnJavaBridgeInit() {
   DCHECK(!java_bridge_dispatcher_.get());
 #if defined(ENABLE_JAVA_BRIDGE)
   java_bridge_dispatcher_.reset(new JavaBridgeDispatcher(this));
 #endif
 }