Move ThreadableLoader to Oilpan heap (2/3)

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

 /*
  * Copyright (C) 2009, 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013, Intel Corporation
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 37 matching lines @@
 
 namespace blink {
 
 class Document;
 class KURL;
 class ResourceRequest;
 class SecurityOrigin;
 class ThreadableLoaderClient;
 
 class CORE_EXPORT DocumentThreadableLoader final : public ThreadableLoader, private RawResourceClient {
-    USING_FAST_MALLOC(DocumentThreadableLoader);
     public:
         static void loadResourceSynchronously(Document&, const ResourceRequest&, ThreadableLoaderClient&, const ThreadableLoaderOptions&, const ResourceLoaderOptions&);
-        static std::unique_ptr<DocumentThreadableLoader> create(Document&, ThreadableLoaderClient*, const ThreadableLoaderOptions&, const ResourceLoaderOptions&);
+        static DocumentThreadableLoader* create(Document&, ThreadableLoaderClient*, const ThreadableLoaderOptions&, const ResourceLoaderOptions&);
         ~DocumentThreadableLoader() override;
 
         void start(const ResourceRequest&) override;
 
         void overrideTimeout(unsigned long timeout) override;
 
         // |this| may be dead after calling this method in async mode.
         void cancel() override;
         void setDefersLoading(bool);
 
+        DECLARE_TRACE();
+
     private:
         enum BlockingBehavior {
             LoadSynchronously,
             LoadAsynchronously
         };
 
         DocumentThreadableLoader(Document&, ThreadableLoaderClient*, BlockingBehavior, const ThreadableLoaderOptions&, const ResourceLoaderOptions&);
 
         void clear();
 
@@ skipping 80 matching lines @@
                 return;
 
             if (RawResource* oldResource = m_resource.release())
                 oldResource->removeClient(this);
 
             if (newResource) {
                 m_resource = newResource;
                 m_resource->addClient(this);
             }
         }
-        Persistent<RawResource> m_resource;
+        Member<RawResource> m_resource;
         // End of ResourceOwner re-implementation, see above.
 
         SecurityOrigin* getSecurityOrigin() const;
         Document& document() const;
 
         ThreadableLoaderClient* m_client;
-        WeakPersistent<Document> m_document;
+        Member<Document> m_document;
 
         const ThreadableLoaderOptions m_options;
         // Some items may be overridden by m_forceDoNotAllowStoredCredentials
         // and m_securityOrigin. In such a case, build a ResourceLoaderOptions
         // with up-to-date values from them and this variable, and use it.
         const ResourceLoaderOptions m_resourceLoaderOptions;
 
         bool m_forceDoNotAllowStoredCredentials;
         RefPtr<SecurityOrigin> m_securityOrigin;
 
@@ skipping 34 matching lines @@
         int m_corsRedirectLimit;
 
         WebURLRequest::FetchRedirectMode m_redirectMode;
 
         // Holds the referrer after a redirect response was
         // received. This referrer is used to populate the HTTP Referer
         // header when following the redirect.
         bool m_didRedirect;
         Referrer m_referrerAfterRedirect;
 
         WeakPtrFactory<DocumentThreadableLoader> m_weakFactory;

[haraken, 2016/07/29 14:52:51]

Now that DocumentThreadableLoader is on the heap, can we remove the weak factory
(in a follow-up CL)?

     };
 
 } // namespace blink
 
 #endif // DocumentThreadableLoader_h