Notifying the Out of Process Renderer about Visibility Change of a Remote Frame

content/browser/renderer_host/render_widget_host_delegate.h

Index: content/browser/renderer_host/render_widget_host_delegate.h
diff --git a/content/browser/renderer_host/render_widget_host_delegate.h b/content/browser/renderer_host/render_widget_host_delegate.h
index b4e4169529bb7e11b1e8da0b392852b01cfdcac1..fce0caa115b165673566640fe8d14a8f91ba74b0 100644
--- a/content/browser/renderer_host/render_widget_host_delegate.h
+++ b/content/browser/renderer_host/render_widget_host_delegate.h
@@ -123,6 +123,10 @@ class CONTENT_EXPORT RenderWidgetHostDelegate {
   // current FrameTree, not including the main frame's SiteInstance.
   virtual void ReplicatePageFocus(bool is_focused) {}
 
+  // If there is a guest WebContents, it should be notified of the visibility
+  // change.
+  virtual void ForwardVisibilityChangeToInnerContents(bool visible) {}

[Charlie Reis, 2015/12/08 19:18:33]

This isn't implemented anywhere.  Is it stale, or do you plan to override it
somewhere?

(We should either implement it now or leave this part out of this CL and add it
when you're ready.)

[EhsanK, 2015/12/09 00:14:16]

I did override it in WebContentsImpl. Purely, this method's purpose is to inform
the out of process renderer of the visibility change and is only valid for
<guestview>'s. That is why I initially left out the implementation to make this
CL only related to oopif.

[Charlie Reis, 2015/12/10 07:45:19]

Leaving it out for now would be fine, but if so we should leave out this
declaration (and the call) as well as the implementation, adding them both
later.

(I'm fine either way at this point.)

+
   // Get the focused RenderWidgetHost associated with |receiving_widget|. A
   // RenderWidgetHostView, upon receiving a keyboard event, will pass its
   // RenderWidgetHost to this function to determine who should ultimately