Add <link rel=preload> onload support for scripts and styles

third_party/WebKit/Source/core/loader/LinkPreloadResourceClients.h

+// 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.
+
+#ifndef LinkPreloadResourceClients_h
+#define LinkPreloadResourceClients_h
+
+#include "core/fetch/FontResource.h"
+#include "core/fetch/ImageResourceClient.h"
+#include "core/fetch/ResourceOwner.h"
+#include "core/fetch/ScriptResource.h"
+#include "core/fetch/StyleSheetResourceClient.h"
+
+namespace blink {
+
+class LinkLoader;
+
+class LinkPreloadResourceClient : public NoBaseWillBeGarbageCollectedFinalized<LinkPreloadResourceClient> {
+public:
+    virtual ~LinkPreloadResourceClient() { }
+
+    void triggerEvents(Resource*);
+
+protected:
+    LinkPreloadResourceClient(LinkLoader* loader)
+        : m_loader(loader)
+    {
+        ASSERT(loader);
+    }
+
+private:
+    LinkLoader* m_loader;
+};
+
+class LinkPreloadScriptResourceClient: public LinkPreloadResourceClient, public ResourceOwner<Resource, ScriptResourceClient> {

[Nate Chapin, 2016/01/15 18:53:58]

Maybe just hold this as a ResourceOwner<ScriptResource> for clarity?

[Yoav Weiss, 2016/01/15 21:40:25]

sure

+public:
+    static PassOwnPtr<LinkPreloadScriptResourceClient> create(LinkLoader* loader, Resource* resource)
+    {
+        if (!loader || !resource)
+            return nullptr;
+        return adoptPtr(new LinkPreloadScriptResourceClient(loader, resource));
+    }
+
+    virtual String debugName() const { return "LinkPreloadScript"; }
+
+    void notifyFinished(Resource* resource) override
+    {
+        ASSERT(this->resource() == resource);
+        triggerEvents(resource);
+    }
+
+private:
+    LinkPreloadScriptResourceClient(LinkLoader* loader, Resource* resource)
+        : LinkPreloadResourceClient(loader)
+    {
+        setResource(resource);
+    }
+};
+
+class LinkPreloadStyleResourceClient: public LinkPreloadResourceClient, public ResourceOwner<Resource, StyleSheetResourceClient> {

[Nate Chapin, 2016/01/15 18:53:58]

Ditto here and CSSStyleSheetResource.

[Yoav Weiss, 2016/01/15 21:40:25]

ok

+public:
+    static PassOwnPtr<LinkPreloadStyleResourceClient> create(LinkLoader* loader, Resource* resource)
+    {
+        if (!loader || !resource)
+            return nullptr;
+        return adoptPtr(new LinkPreloadStyleResourceClient(loader, resource));
+    }
+
+    virtual String debugName() const { return "LinkPreloadStyle"; }
+
+    void notifyFinished(Resource* resource) override

[Nate Chapin, 2016/01/15 18:53:58]

Should this be setCSSStyleSheet()?

[Yoav Weiss, 2016/01/15 21:40:25]

changed

+    {
+        ASSERT(this->resource() == resource);
+        triggerEvents(resource);
+    }
+
+private:
+    LinkPreloadStyleResourceClient(LinkLoader* loader, Resource* resource)
+        : LinkPreloadResourceClient(loader)
+    {
+        setResource(resource);
+    }
+};
+
+}
+
+#endif // LinkPreloadResourceClients_h