Hide fullscreen rotation jank

content/public/common/content_features.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 "build/build_config.h"
 #include "content/public/common/content_features.h"
 
 namespace features {
 
 // All features in alphabetical order.
@@ skipping 91 matching lines @@
 
 // Causes the implementations of guests (inner WebContents) to use
 // out-of-process iframes.
 const base::Feature kGuestViewCrossProcessFrames{
     "GuestViewCrossProcessFrames", base::FEATURE_DISABLED_BY_DEFAULT};
 
 // Enables BlinkGC heap compaction.
 const base::Feature kHeapCompaction{"HeapCompaction",
                                      base::FEATURE_DISABLED_BY_DEFAULT};
 
+// Enables eviction of janky frames that occur on Android during fullscreen
+// transitions.
+const base::Feature kHideFullscreenTransitionJank{
+    "HideFullscreenTransitionJank", base::FEATURE_ENABLED_BY_DEFAULT};

[boliu, 2017/06/20 20:54:44]

is this necessary? are you planning on running an experiment or something?

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

Good question. This was requested by mlamouri@, but has now been removed in the
latest patch, since there are cleanup changes in this CL that would cause
regressions if the feature was disabled. If some people still feel that this
change necessitates a feature flag I can re-add this.

+
 // Enables Blink's idle time spell checker.
 // Design: https://goo.gl/zONC3v
 // Note: The feature is implemented in Blink, and is independent to the
 // ENABLE_SPELLCHECK build flag defined in components/spellcheck.
 const base::Feature kIdleTimeSpellChecking{"IdleTimeSpellChecking",
                                            base::FEATURE_DISABLED_BY_DEFAULT};
 
 // Enables lazily parsing css properties for performance.
 const base::Feature kLazyParseCSS{"LazyParseCSS",
                                   base::FEATURE_DISABLED_BY_DEFAULT};
@@ skipping 251 matching lines @@
 #endif
 
 #if defined(OS_WIN)
 // Emergency "off switch" for new Windows sandbox security mitigation,
 // sandbox::MITIGATION_EXTENSION_POINT_DISABLE.
 const base::Feature kWinSboxDisableExtensionPoints{
     "WinSboxDisableExtensionPoint", base::FEATURE_ENABLED_BY_DEFAULT};
 #endif
 
 }  // namespace features