Move RawResourceClient to Oilpan heap

third_party/WebKit/Source/core/fetch/RawResource.h

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller <mueller@kde.org>
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
     version 2 of the License, or (at your option) any later version.
@@ skipping 84 matching lines @@
     Resource::Type type = resource.getType();
     return type == Resource::MainResource || type == Resource::Raw || type == Resource::TextTrack || type == Resource::Media || type == Resource::Manifest || type == Resource::ImportResource;
 }
 #endif
 inline RawResource* toRawResource(Resource* resource)
 {
     ASSERT_WITH_SECURITY_IMPLICATION(!resource || isRawResource(*resource));
     return static_cast<RawResource*>(resource);
 }
 
-class CORE_EXPORT RawResourceClient : public ResourceClient {
+class CORE_EXPORT RawResourceClient : public GarbageCollectedMixin, public ResourceClient {

[Nate Chapin, 2016/08/03 19:37:01]

Nit: I've usually seen GarbageCollecedMixin as the last declared parent class.
Any particular reason to invert the ordering?

[yhirano, 2016/08/09 07:16:44]

I get an error when I specify GarbageCollecteMixin as the last parent: "error:
[blink-gc] Left-most base class 'ResourceClient' of derived class
'RawResourceClient' must define a virtual trace method."

 public:
-    RawResourceClient()
-        : m_weakFactory(this) { }
-    WeakPtr<RawResourceClient> createWeakPtr() { return m_weakFactory.createWeakPtr(); }
-    ~RawResourceClient() override {}
     static bool isExpectedType(ResourceClient* client) { return client->getResourceClientType() == RawResourceType; }
     ResourceClientType getResourceClientType() const final { return RawResourceType; }
 
     virtual void dataSent(Resource*, unsigned long long /* bytesSent */, unsigned long long /* totalBytesToBeSent */) { }
     virtual void responseReceived(Resource*, const ResourceResponse&, std::unique_ptr<WebDataConsumerHandle>) { }
     virtual void setSerializedCachedMetadata(Resource*, const char*, size_t) { }
     virtual void dataReceived(Resource*, const char* /* data */, size_t /* length */) { }
     virtual void redirectReceived(Resource*, ResourceRequest&, const ResourceResponse&) { }
     virtual void redirectBlocked() {}
     virtual void dataDownloaded(Resource*, int) { }
     virtual void didReceiveResourceTiming(Resource*, const ResourceTimingInfo&) { }
 
-private:
-    WeakPtrFactory<RawResourceClient> m_weakFactory;
+    DEFINE_INLINE_VIRTUAL_TRACE() {}
 };
 
 } // namespace blink
 
 #endif // RawResource_h