[DevTools] Cleanup v8_inspector console API.

third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp

 /*
  * Copyright (C) 2010-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 294 matching lines @@
     , m_instrumentingAgents(m_webLocalFrameImpl->frame()->instrumentingAgents())
     , m_resourceContentLoader(InspectorResourceContentLoader::create(m_webLocalFrameImpl->frame()))
     , m_overlay(overlay)
     , m_inspectedFrames(InspectedFrames::create(m_webLocalFrameImpl->frame()))
     , m_resourceContainer(new InspectorResourceContainer(m_inspectedFrames))
     , m_domAgent(nullptr)
     , m_pageAgent(nullptr)
     , m_networkAgent(nullptr)
     , m_layerTreeAgent(nullptr)
     , m_tracingAgent(nullptr)
-    , m_logAgent(nullptr)
     , m_includeViewAgents(includeViewAgents)
     , m_layerTreeId(0)
 {
     DCHECK(isMainThread());
     DCHECK(m_webLocalFrameImpl->frame());
 }
 
 WebDevToolsAgentImpl::~WebDevToolsAgentImpl()
 {
     DCHECK(!m_client);
@@ skipping 17 matching lines @@
     visitor->trace(m_instrumentingAgents);
     visitor->trace(m_resourceContentLoader);
     visitor->trace(m_overlay);
     visitor->trace(m_inspectedFrames);
     visitor->trace(m_resourceContainer);
     visitor->trace(m_domAgent);
     visitor->trace(m_pageAgent);
     visitor->trace(m_networkAgent);
     visitor->trace(m_layerTreeAgent);
     visitor->trace(m_tracingAgent);
-    visitor->trace(m_logAgent);
     visitor->trace(m_session);
 }
 
 void WebDevToolsAgentImpl::willBeDestroyed()
 {
     DCHECK(m_webLocalFrameImpl->frame());
     DCHECK(m_inspectedFrames->root()->view());
     detach();
     m_resourceContentLoader->dispose();
     m_client = nullptr;
@@ skipping 39 matching lines @@
     m_session->append(tracingAgent);
 
     m_session->append(new InspectorDOMDebuggerAgent(isolate, m_domAgent, m_session->v8Session()));
 
     m_session->append(InspectorInputAgent::create(m_inspectedFrames.get()));
 
     InspectorPageAgent* pageAgent = InspectorPageAgent::create(m_inspectedFrames.get(), this, m_resourceContentLoader.get(), m_session->v8Session());
     m_pageAgent = pageAgent;
     m_session->append(pageAgent);
 
-    m_logAgent = new InspectorLogAgent(&m_inspectedFrames->root()->host()->consoleMessageStorage());
-    m_session->append(m_logAgent.get());
+    m_session->append(new InspectorLogAgent(&m_inspectedFrames->root()->host()->consoleMessageStorage()));
 
     m_tracingAgent->setLayerTreeId(m_layerTreeId);
     m_networkAgent->setHostId(hostId);
 
     if (m_includeViewAgents) {
         // TODO(dgozman): we should actually pass the view instead of frame, but during
         // remote->local transition we cannot access mainFrameImpl() yet, so we have to store the
         // frame which will become the main frame later.
         m_session->append(InspectorRenderingAgent::create(m_webLocalFrameImpl, m_overlay.get()));
         m_session->append(InspectorEmulationAgent::create(m_webLocalFrameImpl, this));
@@ skipping 15 matching lines @@
 void WebDevToolsAgentImpl::destroySession()
 {
     if (m_overlay)
         m_overlay->clear();
 
     m_tracingAgent.clear();
     m_layerTreeAgent.clear();
     m_networkAgent.clear();
     m_pageAgent.clear();
     m_domAgent.clear();
-    m_logAgent.clear();
 
     m_session->dispose();
     m_session.clear();
 
     Platform::current()->currentThread()->removeTaskObserver(this);
 }
 
 void WebDevToolsAgentImpl::attach(const WebString& hostId, int sessionId)
 {
     if (attached())
@@ skipping 142 matching lines @@
     if (m_overlay)
         m_overlay->suspend();
 }
 
 void WebDevToolsAgentImpl::profilingStopped()
 {
     if (m_overlay)
         m_overlay->resume();
 }
 
-void WebDevToolsAgentImpl::consoleCleared()
-{
-    if (m_domAgent)
-        m_domAgent->releaseDanglingNodes();
-    if (m_logAgent)
-        m_logAgent->clear(nullptr);
-}
-
 void WebDevToolsAgentImpl::pageLayoutInvalidated(bool resized)
 {
     if (m_overlay)
         m_overlay->pageLayoutInvalidated(resized);
 }
 
 void WebDevToolsAgentImpl::setPausedInDebuggerMessage(const String& message)
 {
     if (m_overlay)
         m_overlay->setPausedInDebuggerMessage(message);
@@ skipping 56 matching lines @@
 bool WebDevToolsAgent::shouldInterruptForMethod(const WebString& method)
 {
     return method == "Debugger.pause"
         || method == "Debugger.setBreakpoint"
         || method == "Debugger.setBreakpointByUrl"
         || method == "Debugger.removeBreakpoint"
         || method == "Debugger.setBreakpointsActive";
 }
 
 } // namespace blink