Add a new ContentRendererClient::HandleNavigation callback in RenderViewImpl::decidePolicyForNaviga…

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
===================================================================
--- content/renderer/render_view_impl.cc	(revision 165669)
+++ content/renderer/render_view_impl.cc	(working copy)
@@ -2674,6 +2674,12 @@
 WebNavigationPolicy RenderViewImpl::decidePolicyForNavigation(
     WebFrame* frame, const WebURLRequest& request, WebNavigationType type,
     const WebNode&, WebNavigationPolicy default_policy, bool is_redirect) {
+  if (GetContentClient()->renderer()->HandleNavigation(frame, request, type,

[Charlie Reis, 2012/11/08 19:06:12]

We do not want to do this when request.url() == GURL(kSwappedOutURL).  That's an
internal implementation URL that should not be exposed to embedders or the
browser process.

I think adding that to this condition is probably the right thing to do.

[Marshall, 2012/11/08 19:15:59]

Done.

+                                                       default_policy,
+                                                       is_redirect)) {
+    return WebKit::WebNavigationPolicyIgnore;
+  }
+
   Referrer referrer(
       GURL(request.httpHeaderField(WebString::fromUTF8("Referer"))),
       GetReferrerPolicyFromRequest(frame, request));