Oilpan: tidy up InspectorResourceContentLoader's ResourceClient handling.

third_party/WebKit/Source/core/inspector/InspectorResourceContentLoader.h

Index: third_party/WebKit/Source/core/inspector/InspectorResourceContentLoader.h
diff --git a/third_party/WebKit/Source/core/inspector/InspectorResourceContentLoader.h b/third_party/WebKit/Source/core/inspector/InspectorResourceContentLoader.h
index 25c71f8e82d8ed0a1ab4824a21d502f51a8bac2e..ff9ab12643f1cbe503541eabb826050f564e7066 100644
--- a/third_party/WebKit/Source/core/inspector/InspectorResourceContentLoader.h
+++ b/third_party/WebKit/Source/core/inspector/InspectorResourceContentLoader.h
@@ -46,8 +46,8 @@ private:
     bool m_allRequestsStarted;
     bool m_started;
     RawPtrWillBeMember<LocalFrame> m_inspectedFrame;
-    HashSet<ResourceClient*> m_pendingResourceClients;
-    Vector<ResourcePtr<Resource> > m_resources;
+    WillBeHeapHashSet<RawPtrWillBeMember<ResourceClient>> m_pendingResourceClients;

[haraken, 2015/11/13 11:55:04]

Nit: It would be better to make this WillBeHeapHashSet<OwnPtrWillBeMember<>>
(but it's up to you since oilpan will be shipped in the near future).

[sof, 2015/11/13 13:57:27]

The ResourceClients here self-destruct, which they need to do post stop(), so
arranging for them to be owned by InspectorResourceContentLoader will require
too much of a rethink !OILPAN.

+    Vector<ResourcePtr<Resource>> m_resources;
 
     friend class ResourceClient;
 };