Oilpan: Prepare moving InspectorController and InspectorAgents to oilpan.

Source/core/inspector/WorkerInspectorController.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 77 matching lines @@
 
 }
 
 WorkerInspectorController::WorkerInspectorController(WorkerGlobalScope* workerGlobalScope)
     : m_workerGlobalScope(workerGlobalScope)
     , m_stateClient(adoptPtr(new WorkerStateClient(workerGlobalScope)))
     , m_state(adoptPtr(new InspectorCompositeState(m_stateClient.get())))
     , m_instrumentingAgents(InstrumentingAgents::create())
     , m_injectedScriptManager(InjectedScriptManager::createForWorker())
     , m_debugServer(adoptPtr(new WorkerScriptDebugServer(workerGlobalScope)))
-    , m_agents(m_instrumentingAgents.get(), m_state.get())
+    , m_agents(adoptPtrWillBeNoop(new InspectorAgentRegistry(m_instrumentingAgents.get(), m_state.get())))
 {
-    m_agents.append(WorkerRuntimeAgent::create(m_injectedScriptManager.get(), m_debugServer.get(), workerGlobalScope));
+    agents().append(WorkerRuntimeAgent::create(m_injectedScriptManager.get(), m_debugServer.get(), workerGlobalScope));
 
-    OwnPtr<InspectorTimelineAgent> timelineAgent = InspectorTimelineAgent::create(0, 0, 0, InspectorTimelineAgent::WorkerInspector, 0);
-    m_agents.append(WorkerDebuggerAgent::create(m_debugServer.get(), workerGlobalScope, m_injectedScriptManager.get()));
+    OwnPtrWillBeRawPtr<InspectorTimelineAgent> timelineAgent = InspectorTimelineAgent::create(0, 0, 0, InspectorTimelineAgent::WorkerInspector, 0);
+    OwnPtrWillBeRawPtr<WorkerDebuggerAgent> debuggerAgent = WorkerDebuggerAgent::create(m_debugServer.get(), workerGlobalScope, m_injectedScriptManager.get());
+    m_debuggerAgent = debuggerAgent.get();
+    agents().append(debuggerAgent.release());
 
-    m_agents.append(InspectorProfilerAgent::create(m_injectedScriptManager.get(), 0));
-    m_agents.append(InspectorHeapProfilerAgent::create(m_injectedScriptManager.get()));
-    m_agents.append(WorkerConsoleAgent::create(timelineAgent.get(), m_injectedScriptManager.get()));
-    m_agents.append(timelineAgent.release());
+    agents().append(InspectorProfilerAgent::create(m_injectedScriptManager.get(), 0));
+    agents().append(InspectorHeapProfilerAgent::create(m_injectedScriptManager.get()));
+    agents().append(WorkerConsoleAgent::create(timelineAgent.get(), m_injectedScriptManager.get()));
+    agents().append(timelineAgent.release());
 
     m_injectedScriptManager->injectedScriptHost()->init(m_instrumentingAgents.get(), m_debugServer.get());
 }
 
 WorkerInspectorController::~WorkerInspectorController()
 {
-    m_instrumentingAgents->reset();
-    disconnectFrontend();
 }
 
 void WorkerInspectorController::connectFrontend()
 {
     ASSERT(!m_frontend);
     m_state->unmute();
     m_frontendChannel = adoptPtr(new PageInspectorProxy(m_workerGlobalScope));
     m_frontend = adoptPtr(new InspectorFrontend(m_frontendChannel.get()));
     m_backendDispatcher = InspectorBackendDispatcher::create(m_frontendChannel.get());
-    m_agents.registerInDispatcher(m_backendDispatcher.get());
-    m_agents.setFrontend(m_frontend.get());
+    agents().registerInDispatcher(m_backendDispatcher.get());
+    agents().setFrontend(m_frontend.get());
     InspectorInstrumentation::frontendCreated();
 }
 
 void WorkerInspectorController::disconnectFrontend()
 {
     if (!m_frontend)
         return;
     m_backendDispatcher->clearFrontend();
     m_backendDispatcher.clear();
     // Destroying agents would change the state, but we don't want that.
     // Pre-disconnect state will be used to restore inspector agents.
     m_state->mute();
-    m_agents.clearFrontend();
+    agents().clearFrontend();
     m_frontend.clear();
     InspectorInstrumentation::frontendDeleted();
     m_frontendChannel.clear();
 }
 
 void WorkerInspectorController::restoreInspectorStateFromCookie(const String& inspectorCookie)
 {
     ASSERT(!m_frontend);
     connectFrontend();
     m_state->loadFromCookie(inspectorCookie);
 
-    m_agents.restore();
+    agents().restore();
 }
 
 void WorkerInspectorController::dispatchMessageFromFrontend(const String& message)
 {
     if (m_backendDispatcher)
         m_backendDispatcher->dispatch(message);
 }
 
 void WorkerInspectorController::resume()
 {
     if (WorkerRuntimeAgent* runtimeAgent = m_instrumentingAgents->workerRuntimeAgent()) {
         ErrorString unused;
         runtimeAgent->run(&unused);
     }
 }
 
+void WorkerInspectorController::discard()
+{
+    m_debuggerAgent->discard();

[haraken, 2014/06/12 08:40:25]

We conventionally call this kind of method dispose().

[keishi, 2014/06/13 03:37:34]

Done.

+    m_debuggerAgent = nullptr;
+    m_instrumentingAgents->reset();
+    disconnectFrontend();
 }
+
+void WorkerInspectorController::trace(Visitor* visitor)
+{
+    visitor->trace(m_instrumentingAgents);
+    visitor->trace(m_agents);
+    visitor->trace(m_debuggerAgent);
+}
+
+}