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

Unified Diff: content/browser/frame_host/cross_process_frame_connector.cc

Issue 1592573002: Fixing WebViewVisibilityTests for OOPIF-<webview>. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/frame_host/cross_process_frame_connector.cc
diff --git a/content/browser/frame_host/cross_process_frame_connector.cc b/content/browser/frame_host/cross_process_frame_connector.cc
index 1d8b23c041186c75129f4e8b0e6e2c893fe119a8..98d3a6a165d13993d7de8ebaa7bac2457fe1f255 100644
--- a/content/browser/frame_host/cross_process_frame_connector.cc
+++ b/content/browser/frame_host/cross_process_frame_connector.cc
@@ -233,6 +233,17 @@ void CrossProcessFrameConnector::OnVisibilityChanged(bool visible) {
if (!view_)
return;
+ // For <webview>, the WebContents should be notified of the change. The
lazyboy 2016/01/16 00:46:48 Avoid using "<webview>". Use "inner WebContents" i
EhsanK 2016/01/20 00:00:03 Done.
+ // WebContents will inform the RWH itself.
+ if (frame_proxy_in_parent_renderer_->frame_tree_node()
+ ->render_manager()
+ ->ForInnerDelegate()) {
+ RenderWidgetHostImpl::From(view_->GetRenderWidgetHost())
+ ->delegate()
+ ->ForwardVisibilityChange(visible);
+ return;
+ }
+
if (visible)
view_->Show();
else

Powered by Google App Engine
This is Rietveld 408576698