Index: content/browser/renderer_host/render_widget_host_view_base.h |
diff --git a/content/browser/renderer_host/render_widget_host_view_base.h b/content/browser/renderer_host/render_widget_host_view_base.h |
index a56fd81a3d12e362b7a96e6f09aad54354c18c0c..8ed0959a3e7a6c5a7fdee7c62dc6b0b55e665662 100644 |
--- a/content/browser/renderer_host/render_widget_host_view_base.h |
+++ b/content/browser/renderer_host/render_widget_host_view_base.h |
@@ -405,6 +405,9 @@ class CONTENT_EXPORT RenderWidgetHostViewBase : public RenderWidgetHostView, |
// main frame. |
virtual void OnDidNavigateMainFrameToNewPage(); |
+ // Called by the WebContentsImpl when the fullscreen state is changed. |
+ virtual void OnFullscreenStateChanged(bool is_fullscreen) {} |
boliu
2017/06/20 20:54:44
how is this different from the is_fullscreen thing
steimel
2017/06/22 01:32:11
Right now, the is_fullscreen stuff is used in Rend
boliu
2017/06/23 01:51:42
My problem is that this is just too confusing. Hav
Khushal
2017/06/23 03:09:50
Agreed. I think the function will still be needed
steimel
2017/06/29 00:38:50
Changed to not have is_fullscreen param, but as Kh
|
+ |
// Called by WebContentsImpl to notify the view about a change in visibility |
// of context menu. The view can then perform platform specific tasks and |
// changes. |