Commit f7c9df85 authored by Fergal Daly's avatar Fergal Daly Committed by Commit Bot

Make RenderFrameProxyHost::GetRenderViewHost return its own RVH.

RFPH has a render_view_host_ member, introduced in
https://crrev.com/c/1665936 when RVH was made refcounted but that is not
what GetRenderViewHost returns. Instead it pulls it from the frame tree.

Patchset 1 of this CL (https://crrev.com/c/2332109/1) shows that these
values never differ currently, so we can use the simpler (and less
surprising) version.

Change-Id: I2046afd17c66785c63e151f546682c7babf50925
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332109Reviewed-by: default avatarArthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: default avatarAlex Moshchuk <alexmos@chromium.org>
Auto-Submit: Fergal Daly <fergal@chromium.org>
Commit-Queue: Alex Moshchuk <alexmos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#794207}
parent 12fbc5fe
...@@ -172,9 +172,7 @@ void RenderFrameProxyHost::SetChildRWHView( ...@@ -172,9 +172,7 @@ void RenderFrameProxyHost::SetChildRWHView(
} }
RenderViewHostImpl* RenderFrameProxyHost::GetRenderViewHost() { RenderViewHostImpl* RenderFrameProxyHost::GetRenderViewHost() {
return frame_tree_node_->frame_tree() return render_view_host_.get();
->GetRenderViewHost(site_instance_.get())
.get();
} }
RenderWidgetHostView* RenderFrameProxyHost::GetRenderWidgetHostView() { RenderWidgetHostView* RenderFrameProxyHost::GetRenderWidgetHostView() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment