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 581 matching lines @@
     host_->SetViewportSize(size);
   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): Set the color based on the color of the renderer frame.

[aelias_OOO_until_Jul13, 2017/06/16 22:46:40]

This is done in OnFrameMetadataUpdated right now.  Please refactor this so that
there's a unified logic that calls SetBackgroundColor by considering all the
current state instead of having these different subsystems fight each other for
authority over it.

[steimel, 2017/06/22 01:32:11]

Done.

+    SetBackgroundColor(transparent ? SK_ColorBLACK : SK_ColorWHITE);
   }
 }
 
 void CompositorImpl::SetBackgroundColor(int color) {
   host_->set_background_color(color);
 }
 
 void CompositorImpl::SetNeedsComposite() {
   if (!host_->IsVisible())
     return;
@@ skipping 276 matching lines @@
   }
   GetSurfaceManager()->UnregisterFrameSinkHierarchy(frame_sink_id_,
                                                     frame_sink_id);
 }
 
 bool CompositorImpl::HavePendingReadbacks() {
   return !readback_layer_tree_->children().empty();
 }
 
 }  // namespace content