Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to green up headless_…

headless/public/util/protocol_handler_request_id_browsertest.cc

 // Copyright 2017 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 "base/run_loop.h"
+#include "content/public/common/browser_side_navigation_policy.h"
 #include "content/public/test/browser_test.h"
 #include "headless/public/devtools/domains/network.h"
 #include "headless/public/devtools/domains/page.h"
 #include "headless/public/headless_devtools_client.h"
 #include "headless/public/util/expedited_dispatcher.h"
 #include "headless/public/util/generic_url_request_job.h"
 #include "headless/public/util/url_fetcher.h"
 #include "headless/test/headless_browser_test.h"
 #include "net/url_request/url_request_job_factory.h"
 #include "testing/gmock/include/gmock/gmock.h"
@@ skipping 146 matching lines @@
 })";
 
 }  // namespace
 
 class ProtocolHandlerRequestIdCorrelationTest
     : public HeadlessAsyncDevTooledBrowserTest,
       public network::Observer,
       public page::Observer {
  public:
   void RunDevTooledTest() override {
+    if (content::IsBrowserSideNavigationEnabled()) {
+      // TODO: get this working with PlzNavigate.
+      // See discussion on https://codereview.chromium.org/2695923010/
+      FinishAsynchronousTest();
+      return;
+    }
+
     EXPECT_TRUE(embedded_test_server()->Start());
     devtools_client_->GetPage()->AddObserver(this);
     devtools_client_->GetPage()->Enable();
+
+    base::RunLoop run_loop;
     devtools_client_->GetNetwork()->AddObserver(this);
-    devtools_client_->GetNetwork()->Enable();
+    devtools_client_->GetNetwork()->Enable(run_loop.QuitClosure());
+    base::MessageLoop::ScopedNestableTaskAllower nest_loop(
+        base::MessageLoop::current());
+    run_loop.Run();

[nasko, 2017/03/06 18:38:49]

The explanation of why this is needed is at
https://codereview.chromium.org/2720543002/#msg25.

+
     devtools_client_->GetPage()->Navigate("http://foo.com/index.html");
   }
 
   ProtocolHandlerMap GetProtocolHandlers() override {
     ProtocolHandlerMap protocol_handlers;
     std::unique_ptr<RequestIdCorrelationProtocolHandler> http_handler(
         new RequestIdCorrelationProtocolHandler(browser()->BrowserIOThread()));
     http_handler_ = http_handler.get();
     http_handler_->InsertResponse("http://foo.com/index.html",
                                   {kIndexHtml, "text/html"});
@@ skipping 23 matching lines @@
   }
 
  private:
   std::map<std::string, std::string> url_to_devtools_id_;
   RequestIdCorrelationProtocolHandler* http_handler_;  // NOT OWNED
 };
 
 HEADLESS_ASYNC_DEVTOOLED_TEST_F(ProtocolHandlerRequestIdCorrelationTest);
 
 }  // namespace headless