Avoiding going through RenderFrameProxy when targeting a *new* frame.

third_party/WebKit/Source/core/loader/FrameLoader.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2008 Alp Toker <alp@atoker.com>
  * Copyright (C) Research In Motion Limited 2009. All rights reserved.
  * Copyright (C) 2011 Kris Jordan <krisjordan@gmail.com>
  * Copyright (C) 2011 Google Inc. All rights reserved.
@@ skipping 951 matching lines @@
                                   const FrameLoadRequest& request,
                                   NavigationPolicy policy) {
   if (!targetFrame && !request.frameName().isEmpty())
     return true;
   // FIXME: This case is a workaround for the fact that ctrl+clicking a form
   // submission incorrectly sends as a GET rather than a POST if it creates a
   // new window in a different process.
   return request.form() && policy != NavigationPolicyCurrentTab;
 }
 
+static bool shouldNavigateTargetFrame(NavigationPolicy policy) {
+  switch (policy) {
+    case NavigationPolicyCurrentTab:
+      return true;
+
+    // Navigation will target a *new* frame (e.g. because of a ctrl-click),
+    // so the target frame can be ignored.
+    case NavigationPolicyNewBackgroundTab:
+    case NavigationPolicyNewForegroundTab:
+    case NavigationPolicyNewWindow:
+    case NavigationPolicyNewPopup:
+      return false;
+
+    // Navigation won't really target any specific frame,
+    // so the target frame can be ignored.
+    case NavigationPolicyIgnore:
+    case NavigationPolicyDownload:
+      return false;
+
+    case NavigationPolicyHandledByClient:
+      // Impossible, because at this point we shouldn't yet have called
+      // client()->decidePolicyForNavigation(...).
+      NOTREACHED();
+      return true;
+
+    default:
+      NOTREACHED() << policy;
+      return true;
+  }
+}
+
 static NavigationType determineNavigationType(FrameLoadType frameLoadType,
                                               bool isFormSubmission,
                                               bool haveEvent) {
   bool isReload = isReloadLoadType(frameLoadType);
   bool isBackForward = isBackForwardLoadType(frameLoadType);
   if (isFormSubmission)
     return (isReload || isBackForward) ? NavigationTypeFormResubmitted
                                        : NavigationTypeFormSubmitted;
   if (haveEvent)
     return NavigationTypeLinkClicked;
@@ skipping 76 matching lines @@
     return;
   }
 
   FrameLoadRequest request(passedRequest);
   request.resourceRequest().setHasUserGesture(
       UserGestureIndicator::processingUserGesture());
 
   if (!prepareRequestForThisFrame(request))
     return;
 
-  Frame* targetFrame = request.form()
-                           ? nullptr
-                           : m_frame->findFrameForNavigation(
-                                 AtomicString(request.frameName()), *m_frame);
-
   if (isBackForwardLoadType(frameLoadType)) {
     DCHECK(historyItem);
     m_provisionalItem = historyItem;
   }
 
-  if (targetFrame && targetFrame != m_frame) {
+  Frame* targetFrame = request.form()
+                           ? nullptr
+                           : m_frame->findFrameForNavigation(
+                                 AtomicString(request.frameName()), *m_frame);
+  NavigationPolicy policy = navigationPolicyForRequest(request);
+  if (targetFrame && targetFrame != m_frame &&
+      shouldNavigateTargetFrame(policy)) {
     bool wasInSamePage = targetFrame->page() == m_frame->page();
 
     request.setFrameName("_self");
     targetFrame->navigate(request);
     Page* page = targetFrame->page();
     if (!wasInSamePage && page)
       page->chromeClient().focus();
     return;
   }
 
   setReferrerForFrameRequest(request);
 
-  FrameLoadType newLoadType = (frameLoadType == FrameLoadTypeStandard)
-                                  ? determineFrameLoadType(request)
-                                  : frameLoadType;
-  NavigationPolicy policy = navigationPolicyForRequest(request);
   if (shouldOpenInNewWindow(targetFrame, request, policy)) {
     if (policy == NavigationPolicyDownload) {
       client()->loadURLExternally(request.resourceRequest(),
                                   NavigationPolicyDownload, String(), false);
     } else {
       request.resourceRequest().setFrameType(WebURLRequest::FrameTypeAuxiliary);
       createWindowForRequest(request, *m_frame, policy);
     }
     return;
   }
 
   const KURL& url = request.resourceRequest().url();
+  FrameLoadType newLoadType = (frameLoadType == FrameLoadTypeStandard)
+                                  ? determineFrameLoadType(request)
+                                  : frameLoadType;
   bool sameDocumentHistoryNavigation =
       isBackForwardLoadType(newLoadType) &&
       historyLoadType == HistorySameDocumentLoad;
   bool sameDocumentNavigation =
       policy == NavigationPolicyCurrentTab &&
       shouldPerformFragmentNavigation(request.form(),
                                       request.resourceRequest().httpMethod(),
                                       newLoadType, url);
 
   // Perform same document navigation.
@@ skipping 789 matching lines @@
                          m_documentLoader ? m_documentLoader->url() : String());
   return tracedValue;
 }
 
 inline void FrameLoader::takeObjectSnapshot() const {
   TRACE_EVENT_OBJECT_SNAPSHOT_WITH_ID("loading", "FrameLoader", this,
                                       toTracedValue());
 }
 
 }  // namespace blink