Oilpan: Prepare moving InspectorResourceContentLoader to Oilpan

Source/core/inspector/InspectorResourceContentLoader.cpp

 // 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.
 
 #include "config.h"
 #include "core/inspector/InspectorResourceContentLoader.h"
 
 #include "core/FetchInitiatorTypeNames.h"
 #include "core/css/CSSStyleSheet.h"
 #include "core/css/StyleSheetContents.h"
@@ skipping 137 matching lines @@
 void InspectorResourceContentLoader::ensureResourcesContentLoaded(PassOwnPtrWillBeRawPtr<VoidCallback> callback)
 {
     if (!m_started)
         start();
     m_callbacks.append(callback);
     checkDone();
 }
 
 InspectorResourceContentLoader::~InspectorResourceContentLoader()
 {
     stop();

[haraken, 2014/09/02 07:19:00]

stop() touches m_callbacks. We're not allowed to touch on-heap objects in
destructors.

[keishi, 2014/09/02 09:08:51]

Moved cleanup to dispose method.

 }
 
+void InspectorResourceContentLoader::trace(Visitor* visitor)
+{
+    visitor->trace(m_callbacks);
+    visitor->trace(m_page);
+}
+
 void InspectorResourceContentLoader::stop()
 {
     HashSet<ResourceClient*> pendingResourceClients;
     m_pendingResourceClients.swap(pendingResourceClients);
     for (HashSet<ResourceClient*>::const_iterator it = pendingResourceClients.begin(); it != pendingResourceClients.end(); ++it)
         (*it)->m_loader = 0;
     m_resources.clear();
     // Make sure all callbacks are called to prevent infinite waiting time.
     checkDone();
 }
@@ skipping 13 matching lines @@
         (*it)->handleEvent();
 }
 
 void InspectorResourceContentLoader::resourceFinished(ResourceClient* client)
 {
     m_pendingResourceClients.remove(client);
     checkDone();
 }
 
 } // namespace blink