Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1418)

Side by Side Diff: content/browser/devtools/render_view_devtools_agent_host.cc

Issue 341853002: [PowerProfiler] Power profiler service keeps running even after devtools frontend has been closed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@power-profiler-devtools-crash-fix
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/devtools/render_view_devtools_agent_host.h" 5 #include "content/browser/devtools/render_view_devtools_agent_host.h"
6 6
7 #include "base/basictypes.h" 7 #include "base/basictypes.h"
8 #include "base/lazy_instance.h" 8 #include "base/lazy_instance.h"
9 #include "content/browser/child_process_security_policy_impl.h" 9 #include "content/browser/child_process_security_policy_impl.h"
10 #include "content/browser/devtools/devtools_manager_impl.h" 10 #include "content/browser/devtools/devtools_manager_impl.h"
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 const IPC::Message& message) { 159 const IPC::Message& message) {
160 RenderViewDevToolsAgentHost* agent_host = FindAgentHost(source); 160 RenderViewDevToolsAgentHost* agent_host = FindAgentHost(source);
161 return agent_host && agent_host->DispatchIPCMessage(message); 161 return agent_host && agent_host->DispatchIPCMessage(message);
162 } 162 }
163 163
164 RenderViewDevToolsAgentHost::RenderViewDevToolsAgentHost(RenderViewHost* rvh) 164 RenderViewDevToolsAgentHost::RenderViewDevToolsAgentHost(RenderViewHost* rvh)
165 : render_view_host_(NULL), 165 : render_view_host_(NULL),
166 overrides_handler_(new RendererOverridesHandler(this)), 166 overrides_handler_(new RendererOverridesHandler(this)),
167 tracing_handler_( 167 tracing_handler_(
168 new DevToolsTracingHandler(DevToolsTracingHandler::Renderer)), 168 new DevToolsTracingHandler(DevToolsTracingHandler::Renderer)),
169 power_handler_(new DevToolsPowerHandler()), 169 power_handler_(new DevToolsPowerHandler(this)),
170 reattaching_(false) { 170 reattaching_(false) {
171 SetRenderViewHost(rvh); 171 SetRenderViewHost(rvh);
172 DevToolsProtocol::Notifier notifier(base::Bind( 172 DevToolsProtocol::Notifier notifier(base::Bind(
173 &RenderViewDevToolsAgentHost::OnDispatchOnInspectorFrontend, 173 &RenderViewDevToolsAgentHost::OnDispatchOnInspectorFrontend,
174 base::Unretained(this))); 174 base::Unretained(this)));
175 overrides_handler_->SetNotifier(notifier); 175 overrides_handler_->SetNotifier(notifier);
176 tracing_handler_->SetNotifier(notifier); 176 tracing_handler_->SetNotifier(notifier);
177 power_handler_->SetNotifier(notifier); 177 power_handler_->SetNotifier(notifier);
178 g_instances.Get().push_back(this); 178 g_instances.Get().push_back(this);
179 AddRef(); // Balanced in RenderViewHostDestroyed. 179 AddRef(); // Balanced in RenderViewHostDestroyed.
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after
465 465
466 void RenderViewDevToolsAgentHost::OnDispatchOnInspectorFrontend( 466 void RenderViewDevToolsAgentHost::OnDispatchOnInspectorFrontend(
467 const std::string& message) { 467 const std::string& message) {
468 if (!render_view_host_) 468 if (!render_view_host_)
469 return; 469 return;
470 DevToolsManagerImpl::GetInstance()->DispatchOnInspectorFrontend( 470 DevToolsManagerImpl::GetInstance()->DispatchOnInspectorFrontend(
471 this, message); 471 this, message);
472 } 472 }
473 473
474 } // namespace content 474 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698