Hide fullscreen rotation jank

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 40429d51814846c69671b3bce27a1af955e1452b..5bf30334f71e0250c02a6e1f9da6cc496cf11083 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2032,6 +2032,11 @@ void WebContentsImpl::EnterFullscreenMode(const GURL& origin) {
   if (delegate_)
     delegate_->EnterFullscreenModeForTab(this, origin);
 
+  RenderWidgetHostViewBase* rwhvb =
+      static_cast<RenderWidgetHostViewBase*>(GetRenderWidgetHostView());
+  if (rwhvb)
+    rwhvb->OnFullscreenStateChanged(IsFullscreenForCurrentTab());
+
   for (auto& observer : observers_)
     observer.DidToggleFullscreenModeForTab(IsFullscreenForCurrentTab(), false);
 }
@@ -2051,6 +2056,11 @@ void WebContentsImpl::ExitFullscreenMode(bool will_cause_resize) {
     video_view->ExitFullscreen();
 #endif
 
+  RenderWidgetHostViewBase* rwhvb =

[Khushal, 2017/06/15 18:42:56]

Don't we need to call this after delegate_->ExitFullscreenModeForTab on the next
line? Since that is what will actually change the state reported in
IsFullscreenForCurrentTab?

[steimel, 2017/06/15 21:46:39]

Done.

+      static_cast<RenderWidgetHostViewBase*>(GetRenderWidgetHostView());
+  if (rwhvb)
+    rwhvb->OnFullscreenStateChanged(IsFullscreenForCurrentTab());
+
   if (delegate_)
     delegate_->ExitFullscreenModeForTab(this);