Hide fullscreen rotation jank

content/browser/web_contents/web_contents_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/web_contents/web_contents_impl.h"
 
 #include <stddef.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 2014 matching lines @@
   // Make sure any existing fullscreen widget is shut down first.
   RenderWidgetHostView* const widget_view = GetFullscreenRenderWidgetHostView();
   if (widget_view) {
     RenderWidgetHostImpl::From(widget_view->GetRenderWidgetHost())
         ->ShutdownAndDestroyWidget(true);
   }
 
   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);
 }
 
 void WebContentsImpl::ExitFullscreenMode(bool will_cause_resize) {
   // This method is being called to leave renderer-initiated fullscreen mode.
   // Make sure any existing fullscreen widget is shut down first.
   RenderWidgetHostView* const widget_view = GetFullscreenRenderWidgetHostView();
   if (widget_view) {
     RenderWidgetHostImpl::From(widget_view->GetRenderWidgetHost())
         ->ShutdownAndDestroyWidget(true);
   }
 
 #if defined(OS_ANDROID)
   ContentVideoView* video_view = ContentVideoView::GetInstance();
   if (video_view != NULL)
     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);
 
   // The fullscreen state is communicated to the renderer through a resize
   // message. If the change in fullscreen state doesn't cause a view resize
   // then we must ensure web contents exit the fullscreen state by explicitly
   // sending a resize message. This is required for the situation of the browser
   // moving the view into a "browser fullscreen" state and then the contents
   // entering "tab fullscreen". Exiting the contents "tab fullscreen" then won't
   // have the side effect of the view resizing, hence the explicit call here is
@@ skipping 3681 matching lines @@
     new_root->SetOriginalOpener(opener->frame_tree()->root());
 
     if (!opener_suppressed) {
       new_root->SetOpener(opener);
       created_with_opener_ = true;
     }
   }
 }
 
 }  // namespace content