Hide fullscreen rotation jank

third_party/WebKit/Source/web/FullscreenController.cpp

Index: third_party/WebKit/Source/web/FullscreenController.cpp
diff --git a/third_party/WebKit/Source/web/FullscreenController.cpp b/third_party/WebKit/Source/web/FullscreenController.cpp
index c0d5ec4b5b97e793509a5662d21cebcae86047e6..d64c879c41118ade197d3a0e237b318f2f1a3af9 100644
--- a/third_party/WebKit/Source/web/FullscreenController.cpp
+++ b/third_party/WebKit/Source/web/FullscreenController.cpp
@@ -78,6 +78,7 @@ void FullscreenController::DidEnterFullscreen() {
   web_view_base_->SetVisualViewportOffset(FloatPoint());
 
   state_ = State::kFullscreen;
+  web_view_base_->LayerTreeView()->SetIsFullscreen(true);
 
   // Notify all local frames that we have entered fullscreen.
   for (Frame* frame = web_view_base_->GetPage()->MainFrame(); frame;
@@ -266,6 +267,7 @@ void FullscreenController::DidUpdateLayout() {
   // fullscreenElementChanged([..], nullptr) was called while exiting.
 
   state_ = State::kInitial;
+  web_view_base_->LayerTreeView()->SetIsFullscreen(false);

[Khushal, 2017/05/24 02:12:24]

Why not do this in DidExitFullScreen?

[steimel, 2017/05/24 19:32:58]

To me it seemed to make more sense that is_fullscreen becomes false when state_
becomes kInitial as opposed to kNeedsScrollAndStateRestore. WDYT?

[Khushal, 2017/05/25 03:33:40]

The kNeedsScrollAndStateRestore is needed for changes that can happen only after
layout. Just setting the fullscreen state on the LayerTreeHost doesn't need to
wait for that.

[steimel, 2017/06/06 03:07:53]

Done.

 }
 
 void FullscreenController::UpdatePageScaleConstraints(bool remove_constraints) {