OOPIF-ize hung renderers part 2, attempt 2
The embedder needs access to the RenderWidgetHost in order to
fully implement a "hung page" dialog.
BUG=767526
Change-Id: I2d699d96d3fef88be17c54422578c14310826851
Reviewed-on: https://chromium-review.googlesource.com/896343
Reviewed-by: Fady Samuel <[email protected]>
Reviewed-by: Ted Choc <[email protected]>
Reviewed-by: Peter Kasting <[email protected]>
Reviewed-by: Alex Moshchuk <[email protected]>
Commit-Queue: Avi Drissman <[email protected]>
Cr-Commit-Position: refs/heads/master@{#534113}diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 4eda571..4d3cc523 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -5434,10 +5434,8 @@
void WebContentsImpl::RendererUnresponsive(
RenderWidgetHostImpl* render_widget_host) {
- RenderProcessHost* hung_process = render_widget_host->GetProcess();
-
for (auto& observer : observers_)
- observer.OnRendererUnresponsive(hung_process);
+ observer.OnRendererUnresponsive(render_widget_host->GetProcess());
if (ShouldIgnoreUnresponsiveRenderer())
return;
@@ -5446,15 +5444,13 @@
return;
if (delegate_)
- delegate_->RendererUnresponsive(this, hung_process);
+ delegate_->RendererUnresponsive(this, render_widget_host);
}
void WebContentsImpl::RendererResponsive(
RenderWidgetHostImpl* render_widget_host) {
- RenderProcessHost* hung_process = render_widget_host->GetProcess();
-
if (delegate_)
- delegate_->RendererResponsive(this, hung_process);
+ delegate_->RendererResponsive(this, render_widget_host);
}
void WebContentsImpl::BeforeUnloadFiredFromRenderManager(
