Oilpan: tidy up InspectorResourceContentLoader's ResourceClient handling.

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

 // Copyright 2014 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 InspectorResourceContentLoader_h
 #define InspectorResourceContentLoader_h
 
 #include "core/CoreExport.h"
 #include "core/fetch/ResourcePtr.h"
 #include "wtf/Functional.h"
@@ skipping 28 matching lines @@
     void resourceFinished(ResourceClient*);
     void checkDone();
     void start();
     void stop();
     bool hasFinished();
 
     Vector<OwnPtr<Closure>> m_callbacks;
     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;
 };
 
 } // namespace blink
 
 
 #endif // !defined(InspectorResourceContentLoader_h)