PlzNavigate: avoid checking navigation policy twice for render-init requests.

components/test_runner/web_frame_test_client.cc

 // Copyright 2016 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 "components/test_runner/web_frame_test_client.h"
 
 #include <memory>
 
 #include "base/logging.h"
 #include "base/strings/string_piece.h"
@@ skipping 639 matching lines @@
       new_message = new_message.substr(0, file_protocol) +
                     URLSuitableForTestResult(new_message.substr(file_protocol));
     }
     delegate_->PrintMessage(new_message);
   }
   delegate_->PrintMessage(std::string("\n"));
 }
 
 blink::WebNavigationPolicy WebFrameTestClient::decidePolicyForNavigation(
     const blink::WebFrameClient::NavigationPolicyInfo& info) {
+  // PlzNavigate
+  // Navigation requests initiated by the renderer have checked navigation
+  // policy when the navigation was sent to the browser. Some layout tests
+  // expect that navigation policy is only checked once.
+  if (delegate_->IsNavigationInitiatedByRenderer(info.urlRequest))
+    return info.defaultPolicy;
+
   if (test_runner()->shouldDumpNavigationPolicy()) {
     delegate_->PrintMessage("Default policy for navigation to '" +
                             URLDescription(info.urlRequest.url()) + "' is '" +
                             WebNavigationPolicyToString(info.defaultPolicy) +
                             "'\n");
   }
 
   blink::WebNavigationPolicy result;
   if (!test_runner()->policyDelegateEnabled())
     return info.defaultPolicy;
@@ skipping 46 matching lines @@
 blink::WebEffectiveConnectionType
 WebFrameTestClient::getEffectiveConnectionType() {
   return test_runner()->effective_connection_type();
 }
 
 TestRunner* WebFrameTestClient::test_runner() {
   return web_view_test_proxy_base_->test_interfaces()->GetTestRunner();
 }
 
 }  // namespace test_runner