Declarative resource hints go through mojo IPC to //content

third_party/WebKit/Source/core/loader/LinkLoaderTest.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/loader/LinkLoader.h"
 
 #include "core/fetch/MemoryCache.h"
 #include "core/fetch/ResourceFetcher.h"
 #include "core/frame/Settings.h"
 #include "core/html/LinkRelAttribute.h"
 #include "core/loader/DocumentLoader.h"
 #include "core/loader/LinkLoaderClient.h"
-#include "core/loader/NetworkHintsInterface.h"
 #include "core/testing/DummyPageHolder.h"
 #include "platform/network/ResourceLoadPriority.h"
+#include "platform/testing/PlatformMojoMock.h"
 #include "platform/testing/URLTestHelpers.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebURLLoaderMockFactory.h"
+#include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include <base/macros.h>
 
 namespace blink {
 
+using ::testing::_;
+
 class MockLinkLoaderClient final : public GarbageCollectedFinalized<MockLinkLoaderClient>, public LinkLoaderClient {
     USING_GARBAGE_COLLECTED_MIXIN(MockLinkLoaderClient);
 public:
     static MockLinkLoaderClient* create(bool shouldLoad)
     {
         return new MockLinkLoaderClient(shouldLoad);
     }
 
     DEFINE_INLINE_VIRTUAL_TRACE() { LinkLoaderClient::trace(visitor); }
 
@@ skipping 11 matching lines @@
 
 private:
     explicit MockLinkLoaderClient(bool shouldLoad)
         : m_shouldLoad(shouldLoad)
     {
     }
 
     bool m_shouldLoad;
 };
 
-class NetworkHintsMock : public NetworkHintsInterface {
-public:
-    NetworkHintsMock()
-        : m_didDnsPrefetch(false)
-        , m_didPreconnect(false)
-    {
-    }
-
-    void dnsPrefetchHost(const String& host) const override
-    {
-        m_didDnsPrefetch = true;
-    }
-
-    void preconnectHost(const KURL& host, const CrossOriginAttributeValue crossOrigin) const override
-    {
-        m_didPreconnect = true;
-        m_isHTTPS = host.protocolIs("https");
-        m_isCrossOrigin = (crossOrigin == CrossOriginAttributeAnonymous);
-    }
-
-    bool didDnsPrefetch() { return m_didDnsPrefetch; }
-    bool didPreconnect() { return m_didPreconnect; }
-    bool isHTTPS() { return m_isHTTPS; }
-    bool isCrossOrigin() { return m_isCrossOrigin; }
-
-private:
-    mutable bool m_didDnsPrefetch;
-    mutable bool m_didPreconnect;
-    mutable bool m_isHTTPS;
-    mutable bool m_isCrossOrigin;
-
-};
-
 TEST(LinkLoaderTest, Preload)
 {
     struct TestCase {
         const char* href;
         const char* as;
         const char* type;
         const char* media;
         const ResourceLoadPriority priority;
         const WebURLRequest::RequestContext context;
         const bool linkLoaderShouldLoadValue;
@@ skipping 42 matching lines @@
         Persistent<MockLinkLoaderClient> loaderClient = MockLinkLoaderClient::create(testCase.linkLoaderShouldLoadValue);
         LinkLoader* loader = LinkLoader::create(loaderClient.get());
         KURL hrefURL = KURL(KURL(), testCase.href);
         URLTestHelpers::registerMockedErrorURLLoad(hrefURL);
         loader->loadLink(LinkRelAttribute("preload"),
             CrossOriginAttributeNotSet,
             testCase.type,
             testCase.as,
             testCase.media,
             hrefURL,
-            dummyPageHolder->document(),
-            NetworkHintsMock());
+            dummyPageHolder->document());
         ASSERT(dummyPageHolder->document().fetcher());
         HeapListHashSet<Member<Resource>>* preloads = dummyPageHolder->document().fetcher()->preloads();
         if (testCase.expectingLoad) {
             if (!preloads)
                 fprintf(stderr, "Unexpected result %s %s %s\n", testCase.href, testCase.as, testCase.type);
             ASSERT_NE(nullptr, preloads);
         } else {
             ASSERT_EQ(nullptr, preloads);
         }
         if (preloads) {
@@ skipping 27 matching lines @@
 
     // TODO(yoav): Test (and fix) shouldLoad = false
 
     // Test the cases with a single header
     for (const auto& testCase : cases) {
         OwnPtr<DummyPageHolder> dummyPageHolder = DummyPageHolder::create(IntSize(500, 500));
         dummyPageHolder->document().settings()->setDNSPrefetchingEnabled(true);
         Persistent<MockLinkLoaderClient> loaderClient = MockLinkLoaderClient::create(testCase.shouldLoad);
         LinkLoader* loader = LinkLoader::create(loaderClient.get());
         KURL hrefURL = KURL(KURL(ParsedURLStringTag(), String("http://example.com")), testCase.href);
-        NetworkHintsMock networkHints;
+
+        PlatformMojoMock mockPlatform;
+        EXPECT_CALL(mockPlatform, preconnect(_, _, _))
+            .Times(0);
+        EXPECT_CALL(mockPlatform, preresolve(hrefURL))
+            .Times(1);
+
         loader->loadLink(LinkRelAttribute("dns-prefetch"),
             CrossOriginAttributeNotSet,
             String(),
             String(),
             String(),
             hrefURL,
-            dummyPageHolder->document(),
-            networkHints);
-        ASSERT_FALSE(networkHints.didPreconnect());
-        ASSERT_EQ(testCase.shouldLoad, networkHints.didDnsPrefetch());

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

Again, where did the asserts go?

+            dummyPageHolder->document());
     }
 }
 
 TEST(LinkLoaderTest, Preconnect)
 {
     struct {
         const char* href;
         CrossOriginAttributeValue crossOrigin;
         const bool shouldLoad;
-        const bool isHTTPS;
-        const bool isCrossOrigin;
     } cases[] = {
-        {"http://example.com/", CrossOriginAttributeNotSet, true, false, false},
-        {"https://example.com/", CrossOriginAttributeNotSet, true, true, false},
-        {"http://example.com/", CrossOriginAttributeAnonymous, true, false, true},
-        {"//example.com/", CrossOriginAttributeNotSet, true, false, false},
+        {"http://example.com/", CrossOriginAttributeNotSet, true},
+        {"https://example.com/", CrossOriginAttributeNotSet, true},
+        {"http://example.com/", CrossOriginAttributeAnonymous, true},
+        {"//example.com/", CrossOriginAttributeNotSet, true},
     };
 
     // Test the cases with a single header
     for (const auto& testCase : cases) {
         OwnPtr<DummyPageHolder> dummyPageHolder = DummyPageHolder::create(IntSize(500, 500));
         Persistent<MockLinkLoaderClient> loaderClient = MockLinkLoaderClient::create(testCase.shouldLoad);
         LinkLoader* loader = LinkLoader::create(loaderClient.get());
         KURL hrefURL = KURL(KURL(ParsedURLStringTag(), String("http://example.com")), testCase.href);
-        NetworkHintsMock networkHints;
+
+        PlatformMojoMock mockPlatform;
+        EXPECT_CALL(mockPlatform, preconnect(hrefURL, ::testing::Eq(testCase.crossOrigin != CrossOriginAttributeAnonymous), 1))
+            .Times(1);
+        EXPECT_CALL(mockPlatform, preresolve(_))
+            .Times(0);
+
         loader->loadLink(LinkRelAttribute("preconnect"),
             testCase.crossOrigin,
             String(),
             String(),
             String(),
             hrefURL,
-            dummyPageHolder->document(),
-            networkHints);
-        ASSERT_EQ(testCase.shouldLoad, networkHints.didPreconnect());
-        ASSERT_EQ(testCase.isHTTPS, networkHints.isHTTPS());
-        ASSERT_EQ(testCase.isCrossOrigin, networkHints.isCrossOrigin());
+            dummyPageHolder->document());
     }
 }
 
 } // namespace blink