Declarative resource hints go through mojo IPC to //content

chrome/browser/net/predictor_browsertest.cc

 // Copyright (c) 2012 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 <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
 #include <memory>
 #include <set>
@@ skipping 18 matching lines @@
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "components/prefs/pref_service.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/test_utils.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/ip_endpoint.h"
 #include "net/base/net_errors.h"
-#include "net/dns/host_resolver_proc.h"
 #include "net/dns/mock_host_resolver.h"
 #include "net/http/http_transaction_factory.h"
 #include "net/socket/stream_socket.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 #include "net/test/embedded_test_server/embedded_test_server_connection_listener.h"
 #include "net/test/embedded_test_server/http_request.h"
 #include "net/test/embedded_test_server/http_response.h"
 #include "net/test/url_request/url_request_failed_job.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
@@ skipping 41 matching lines @@
   if (request.GetURL().path() != path)
     return nullptr;
   std::unique_ptr<net::test_server::BasicHttpResponse> response(
       new net::test_server::BasicHttpResponse);
   response->set_code(net::HTTP_MOVED_PERMANENTLY);
   response->AddCustomHeader("Location", redirect_url.spec());
   return std::move(response);
 }
 
 const char kBlinkPreconnectFeature[] = "LinkPreconnect";
-const char kChromiumUrl[] = "http://chromium.org";
-const char kInvalidLongUrl[] =
-    "http://"
-    "illegally-long-hostname-over-255-characters-should-not-send-an-ipc-"
-    "message-to-the-browser-"
-    "00000000000000000000000000000000000000000000000000000000000000000000000000"
-    "00000000000000000000000000000000000000000000000000000000000000000000000000"
-    "0000000000000000000000000000000000000000000000000000.org";
 
 // Gets notified by the EmbeddedTestServer on incoming connections being
 // accepted or read from, keeps track of them and exposes that info to
 // the tests.
 // A port being reused is currently considered an error.  If a test
 // needs to verify multiple connections are opened in sequence, that will need
 // to be changed.
 class ConnectionListener
     : public net::test_server::EmbeddedTestServerConnectionListener {
  public:
@@ skipping 1259 matching lines @@
   // inspecting that the Predictor starts making the expected hostname requests.
   PrepareFrameSubresources(referring_url_);
   observer()->WaitUntilHostLookedUp(target_url_);
 
   // Verify that both urls were requested by the predictor. Note that the
   // startup URL may be requested before the observer attaches itself.
   ExpectUrlRequestedFromPredictor(startup_url_);
   EXPECT_FALSE(observer()->HostFound(target_url_));
 }
 
-IN_PROC_BROWSER_TEST_F(PredictorBrowserTest, DnsPrefetch) {
-  // Navigate once to make sure all initial hostnames are requested.
-  ui_test_utils::NavigateToURL(browser(),
-                               embedded_test_server()->GetURL("/title1.html"));
-
-  size_t hosts_looked_up_before_load = observer()->TotalHostsLookedUp();
-
-  ui_test_utils::NavigateToURL(browser(), embedded_test_server()->GetURL(
-                                              "/predictor/dns_prefetch.html"));
-  observer()->WaitUntilHostLookedUp(GURL(kChromiumUrl));
-  ASSERT_FALSE(observer()->HasHostBeenLookedUp(GURL(kInvalidLongUrl)));
-
-  EXPECT_FALSE(observer()->HostFound(GURL(kChromiumUrl)));
-  ASSERT_EQ(hosts_looked_up_before_load + 1, observer()->TotalHostsLookedUp());
-}
-
 // Tests that preconnect warms up a socket connection to a test server.
 // Note: This test uses a data URI to serve the preconnect hint, to make sure
 // that the network stack doesn't just re-use its connection to the test server.
+// TODO(csharrison): Move these tests to //content/browser/loader. Now that
+// resource hints have migrated to live there. This might require sharing the
+// preconnect test harness logic.
 IN_PROC_BROWSER_TEST_F(PredictorBrowserTest, PreconnectNonCORS) {
   GURL preconnect_url = embedded_test_server()->base_url();
   std::string preconnect_content =
       "<link rel=\"preconnect\" href=\"" + preconnect_url.spec() + "\">";
   NavigateToDataURLWithContent(preconnect_content);
   connection_listener_->WaitUntilFirstConnectionAccepted();
   EXPECT_EQ(1u, connection_listener_->GetAcceptedSocketCount());
   EXPECT_EQ(0u, connection_listener_->GetReadSocketCount());
 }
 
@@ skipping 85 matching lines @@
   // Second navigation to content with an img.
   std::string img_content =
       "<img src=\"" + preconnect_url.spec() + "test.gif\">";
   NavigateToDataURLWithContent(img_content);
   connection_listener_->WaitUntilFirstConnectionRead();
   EXPECT_EQ(2u, connection_listener_->GetAcceptedSocketCount());
   EXPECT_EQ(1u, connection_listener_->GetReadSocketCount());
 }
 
 }  // namespace chrome_browser_net