Could you please take a look?
Patch set 2:Commit-Queue +1
To view, visit change 830977. To unsubscribe, or for help writing mail filters, visit settings.
Patch set 2:Code-Review +1
Patch set 2:Commit-Queue +2
Commit Bot merged this change.
[DevTools] Stop listening to WidgetWillClose
It was introduced to resume other pages in the same render process
when the page paused on breakpoint is closed.
This now works due to Mojo connection error handler triggering
detach automatically and unpausing as a result.
Removing content-specific code from DevToolsAgent unblocks
migrating it to blink.
Bug: 776009
Change-Id: I20113df99a947eeaeabfb59886a07fc3f866ea27
Reviewed-on: https://chromium-review.googlesource.com/830977
Reviewed-by: Pavel Feldman <pfel...@chromium.org>
Commit-Queue: Dmitry Gozman <dgo...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524541}
---
M content/renderer/devtools/devtools_agent.cc
M content/renderer/devtools/devtools_agent.h
M content/renderer/render_view_browsertest.cc
A third_party/WebKit/LayoutTests/http/tests/inspector-protocol/target/resume-on-close-expected.txt
A third_party/WebKit/LayoutTests/http/tests/inspector-protocol/target/resume-on-close.js
5 files changed, 50 insertions(+), 23 deletions(-)