PlzNavigate: make ctrl+click works.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index cefdf2ba5537e1728dde9898f84d1cdb75309bea..21bb0387a47b4636b2a3ee21361203be104a41c8 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -5086,7 +5086,8 @@ WebNavigationPolicy RenderFrameImpl::decidePolicyForNavigation(
   // This includes navigations with no request being sent to the network stack.
   if (IsBrowserSideNavigationEnabled() &&
       info.urlRequest.checkForBrowserSideNavigation() &&
-      ShouldMakeNetworkRequestForURL(url)) {
+      ShouldMakeNetworkRequestForURL(url) &&
+      info.defaultPolicy == blink::WebNavigationPolicyCurrentTab) {

[clamy, 2016/09/09 15:28:01]

I think we should still have a special case for PlzNavigate all the time, but
set up like this:
if (IsBrowserSideNavigationEnabled() &&
    info.urlRequest.checkForBrowserSideNavigation()) {
  if (info.defaultPolicy == blink::WebNavigationPolicyCurrentTab) {
    BeginNavigation(info);
    return blink::WebNavigationPolicyHandledByClient;
  } else {
    loadURLExternally(...);
    return either WebNavigationPolicyHandledByClient or
WebNavigationPolicyIgnore
  }
}

     BeginNavigation(info);
     return blink::WebNavigationPolicyHandledByClient;
   }