Hide fullscreen rotation jank

content/browser/renderer_host/compositor_impl_android.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/renderer_host/compositor_impl_android.h"
 
 #include <android/bitmap.h>
 #include <android/native_window_jni.h>
 #include <stdint.h>
 #include <unordered_set>
@@ skipping 41 matching lines @@
 #include "components/viz/service/display_compositor/host_shared_bitmap_manager.h"
 #include "components/viz/service/frame_sinks/frame_sink_manager_impl.h"
 #include "content/browser/compositor/surface_utils.h"
 #include "content/browser/gpu/browser_gpu_channel_host_factory.h"
 #include "content/browser/gpu/browser_gpu_memory_buffer_manager.h"
 #include "content/browser/gpu/compositor_util.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/common/gpu_stream_constants.h"
 #include "content/public/browser/android/compositor.h"
 #include "content/public/browser/android/compositor_client.h"
+#include "content/public/common/content_features.h"
 #include "content/public/common/content_switches.h"
 #include "gpu/command_buffer/client/context_support.h"
 #include "gpu/command_buffer/client/gles2_interface.h"
 #include "gpu/ipc/client/command_buffer_proxy_impl.h"
 #include "gpu/ipc/client/gpu_channel_host.h"
 #include "gpu/ipc/common/gpu_surface_tracker.h"
 #include "gpu/vulkan/features.h"
 #include "gpu/vulkan/vulkan_surface.h"
 #include "services/ui/public/cpp/gpu/context_provider_command_buffer.h"
 #include "third_party/khronos/GLES2/gl2.h"
@@ skipping 523 matching lines @@
   if (display_)
     display_->Resize(size);
   root_window_->GetLayer()->SetBounds(size);
 }
 
 void CompositorImpl::SetHasTransparentBackground(bool transparent) {
   has_transparent_background_ = transparent;
   if (host_) {
     host_->set_has_transparent_background(transparent);
 
-    // Give a delay in setting the background color to avoid the color for
-    // the normal mode (white) affecting the UI transition.
-    base::ThreadTaskRunnerHandle::Get().get()->PostDelayedTask(
-        FROM_HERE,
-        base::Bind(&CompositorImpl::SetBackgroundColor,
-                   weak_factory_.GetWeakPtr(),
-                   transparent ? SK_ColorBLACK : SK_ColorWHITE),
-        base::TimeDelta::FromMilliseconds(500));
+    // TODO(steimel): Remove this feature flag and the hack inside this if block
+    // once we're confident that the fullscreen transition hiding is working
+    // properly.
+    if (!base::FeatureList::IsEnabled(
+            features::kHideFullscreenTransitionJank)) {
+      base::ThreadTaskRunnerHandle::Get().get()->PostDelayedTask(
+          FROM_HERE,
+          base::Bind(&CompositorImpl::SetBackgroundColor,
+                     weak_factory_.GetWeakPtr(),
+                     transparent ? SK_ColorBLACK : SK_ColorWHITE),
+          base::TimeDelta::FromMilliseconds(500));
+    }
   }
 }
 
 void CompositorImpl::SetBackgroundColor(int color) {
   host_->set_background_color(color);
 }
 
 void CompositorImpl::SetNeedsComposite() {
   if (!host_->IsVisible())
     return;
@@ skipping 283 matching lines @@
   }
   GetSurfaceManager()->UnregisterFrameSinkHierarchy(frame_sink_id_,
                                                     frame_sink_id);
 }
 
 bool CompositorImpl::HavePendingReadbacks() {
   return !readback_layer_tree_->children().empty();
 }
 
 }  // namespace content