Declarative resource hints go through mojo IPC to //content

third_party/WebKit/Source/core/html/parser/HTMLResourcePreloaderTest.cpp

 // Copyright 2015 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 "core/html/parser/HTMLResourcePreloader.h"
 
 #include "core/html/parser/PreloadRequest.h"
 #include "core/testing/DummyPageHolder.h"
+#include "platform/testing/PlatformMojoMock.h"
+#include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace blink {
 
+using ::testing::_;
+
 struct PreconnectTestCase {
     const char* baseURL;
     const char* url;
     bool isCORS;
     bool isHTTPS;
 };
 
-class PreloaderNetworkHintsMock : public NetworkHintsInterface {
-public:
-    PreloaderNetworkHintsMock()
-        : m_didPreconnect(false)
-    {
-    }
-
-    void dnsPrefetchHost(const String& host) const { }
-    void preconnectHost(const KURL& host, const CrossOriginAttributeValue crossOrigin) const override
-    {
-        m_didPreconnect = true;
-        m_isHTTPS = host.protocolIs("https");
-        m_isCrossOrigin = (crossOrigin == CrossOriginAttributeAnonymous);
-    }
-
-    bool didPreconnect() { return m_didPreconnect; }
-    bool isHTTPS() { return m_isHTTPS; }
-    bool isCrossOrigin() { return m_isCrossOrigin; }
-
-private:
-    mutable bool m_didPreconnect;
-    mutable bool m_isHTTPS;
-    mutable bool m_isCrossOrigin;
-};
-
 class HTMLResourcePreloaderTest : public testing::Test {
 protected:
     HTMLResourcePreloaderTest()
         : m_dummyPageHolder(DummyPageHolder::create())
     {
     }
 
     void test(PreconnectTestCase testCase)
     {
         // TODO(yoav): Need a mock loader here to verify things are happenning beyond preconnect.
-        PreloaderNetworkHintsMock networkHints;
+        KURL hrefURL = KURL(KURL(ParsedURLStringTag(), String(testCase.baseURL)), testCase.url);
+
+        PlatformMojoMock mockPlatform;
+        EXPECT_CALL(mockPlatform, preconnect(hrefURL, ::testing::Ne(testCase.isCORS), 1))
+            .Times(1);
+        EXPECT_CALL(mockPlatform, preresolve(_))
+            .Times(0);
+
         OwnPtr<PreloadRequest> preloadRequest = PreloadRequest::create(String(),
             TextPosition(),
             testCase.url,
             KURL(ParsedURLStringTag(), testCase.baseURL),
             Resource::Image,
             ReferrerPolicy(),
             FetchRequest::ResourceWidth(),
             ClientHintsPreferences(),
             PreloadRequest::RequestTypePreconnect);
         if (testCase.isCORS)
             preloadRequest->setCrossOrigin(CrossOriginAttributeAnonymous);
         HTMLResourcePreloader* preloader = HTMLResourcePreloader::create(m_dummyPageHolder->document());
-        preloader->preload(std::move(preloadRequest), networkHints);
-        ASSERT_TRUE(networkHints.didPreconnect());
-        ASSERT_EQ(testCase.isCORS, networkHints.isCrossOrigin());
-        ASSERT_EQ(testCase.isHTTPS, networkHints.isHTTPS());
+        preloader->preload(std::move(preloadRequest));

[Yoav Weiss, 2016/06/13 08:26:08]

Where are we verifying that we're actually preconnecting with the right CORS
mode and protocol?

[Charlie Harrison, 2016/06/14 19:28:45]

The protocol is included in the EXPECT_CALL with the right url. I thought that
would be enough because if the urls match, the protocol should match.

CORS mode is checks as the second argument, where we pass cors !=
CrossOriginAnonymous (credentials flag) into the preconnect call. This is the
opposite of isCors, so we use testing::Ne here.

The latter could probably be improved, but I was saving it for a follow up patch
where we move the CrossOriginAttributeValue into platform to become part of this
API.

     }
 
 private:
     OwnPtr<DummyPageHolder> m_dummyPageHolder;
 };
 
 TEST_F(HTMLResourcePreloaderTest, testPreconnect)
 {
     PreconnectTestCase testCases[] = {
         { "http://example.test", "http://example.com", false, false },
         { "http://example.test", "http://example.com", true, false },
         { "http://example.test", "https://example.com", true, true },
         { "http://example.test", "https://example.com", false, true },
         { "http://example.test", "//example.com", false, false },
         { "http://example.test", "//example.com", true, false },
         { "https://example.test", "//example.com", false, true },
         { "https://example.test", "//example.com", true, true },
     };
 
     for (const auto& testCase : testCases)
         test(testCase);
 }
 
 } // namespace blink