Hide fullscreen rotation jank

chrome/browser/android/compositor/compositor_view.cc

 // Copyright 2015 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 "chrome/browser/android/compositor/compositor_view.h"
 
 #include <android/bitmap.h>
 #include <android/native_window_jni.h>
 
 #include <memory>
@@ skipping 167 matching lines @@
   gfx::Size size(width, height);
   web_contents->GetNativeView()->OnPhysicalBackingSizeChanged(size);
 }
 
 void CompositorView::SetLayoutBounds(JNIEnv* env,
                                      const JavaParamRef<jobject>& object) {
   root_layer_->SetBounds(gfx::Size(content_width_, content_height_));
 }
 
 void CompositorView::SetBackground(bool visible, SkColor color) {
-  if (overlay_video_mode_)
-    visible = false;
   root_layer_->SetBackgroundColor(color);
   root_layer_->SetIsDrawable(visible);
+  compositor_->SetHasTransparentBackground(SkColorGetA(color) == SK_AlphaTRANSPARENT);

[Khushal, 2017/05/31 20:26:51]

Can you also remove the hack in CompositorImpl?

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

Done.

 }
 
 void CompositorView::SetOverlayVideoMode(JNIEnv* env,
                                          const JavaParamRef<jobject>& object,
                                          bool enabled) {
   if (overlay_video_mode_ == enabled)
     return;
   overlay_video_mode_ = enabled;
-  compositor_->SetHasTransparentBackground(enabled);
   SetNeedsComposite(env, object);
 }
 
 void CompositorView::SetSceneLayer(JNIEnv* env,
                                    const JavaParamRef<jobject>& object,
                                    const JavaParamRef<jobject>& jscene_layer) {
   SceneLayer* scene_layer = SceneLayer::FromJavaObject(env, jscene_layer);
 
   if (scene_layer_ != scene_layer) {
     // The old tree should be detached only if it is not the cached layer or
@@ skipping 59 matching lines @@
   // The Android TERMINATION_STATUS_OOM_PROTECTED hack causes us to never go
   // through here but through BrowserChildProcessHostDisconnected() instead.
 }
 
 // Register native methods
 bool RegisterCompositorView(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace android