DevTools: force-detach worker sessions on navigation
When renderer navigates, all worker agent hosts will be eventually
disposed. In order to keep protocol determenistic, this patch proactively
detaches all worker sessions on renderer navigation.
R=dgozman
Change-Id: Ief0fe3db90eade9e12baa3cbe4f6b78785c60e4f
Reviewed-on: https://chromium-review.googlesource.com/c/1345529
Commit-Queue: Andrey Lushnikov <[email protected]>
Reviewed-by: Dmitry Gozman <[email protected]>
Cr-Commit-Position: refs/heads/master@{#609886}diff --git a/content/browser/devtools/devtools_renderer_channel.h b/content/browser/devtools/devtools_renderer_channel.h
index 10ec64a..6b369b9 100644
--- a/content/browser/devtools/devtools_renderer_channel.h
+++ b/content/browser/devtools/devtools_renderer_channel.h
@@ -22,6 +22,7 @@
class DevToolsAgentHostImpl;
class DevToolsSession;
class RenderFrameHostImpl;
+class WorkerDevToolsAgentHost;
namespace protocol {
class TargetAutoAttacher;
@@ -81,7 +82,7 @@
RenderFrameHostImpl* frame_host_ = nullptr;
base::flat_set<protocol::TargetAutoAttacher*> report_attachers_;
base::flat_set<protocol::TargetAutoAttacher*> wait_for_debugger_attachers_;
- base::flat_set<DevToolsAgentHostImpl*> child_workers_;
+ base::flat_set<WorkerDevToolsAgentHost*> child_workers_;
base::WeakPtrFactory<DevToolsRendererChannel> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(DevToolsRendererChannel);
