Oilpan: Prepare moving InspectorController and InspectorAgents to oilpan.

Source/core/inspector/InspectorResourceAgent.cpp

Index: Source/core/inspector/InspectorResourceAgent.cpp
diff --git a/Source/core/inspector/InspectorResourceAgent.cpp b/Source/core/inspector/InspectorResourceAgent.cpp
index c0a886b5457ee3545a02aff620db9128044ef3ae..db58d30f41ea2a9401669ef97f57b093d5348f24 100644
--- a/Source/core/inspector/InspectorResourceAgent.cpp
+++ b/Source/core/inspector/InspectorResourceAgent.cpp
@@ -286,11 +286,19 @@ static PassRefPtr<TypeBuilder::Network::Response> buildObjectForResourceResponse
 
 InspectorResourceAgent::~InspectorResourceAgent()
 {
+#if !ENABLE(OILPAN)
     if (m_state->getBoolean(ResourceAgentState::resourceAgentEnabled)) {

[haraken, 2014/06/17 14:06:17]

Removing the line 290 - 294 is fine in oilpan builds, since m_state and
m_instrumentingAgents die together with the InspectorResourceAgent.

However, I guess we can also remove the line 290 - 294 in non-oilpan builds as
well. Since clearFrontEnd() is already called, we won't need to call disable()
in the destructor. Not related to your CL though.

         ErrorString error;
         disable(&error);
     }
     ASSERT(!m_instrumentingAgents->inspectorResourceAgent());
+#endif
+}
+
+void InspectorResourceAgent::trace(Visitor* visitor)
+{
+    visitor->trace(m_pageAgent);
+    InspectorBaseAgent::trace(visitor);
 }
 
 void InspectorResourceAgent::willSendRequest(unsigned long identifier, DocumentLoader* loader, ResourceRequest& request, const ResourceResponse& redirectResponse, const FetchInitiatorInfo& initiatorInfo)