Remove feature flag for layout transition tag cleanup

Summary:
Removes stale feature flag that was in production for a couple of months. Fix helped to decrease number of crashes significantly, so we can remove it.

Changelog: [Internal]

Reviewed By: mdvacca

Differential Revision: D28757995

fbshipit-source-id: 375da09c11f265e8bbe03cd99de1b83f168420ce
This commit is contained in:
Andrei Shikov
2021-05-27 14:43:53 -07:00
committed by Facebook GitHub Bot
parent 8a5ee96354
commit d1ab03235c
3 changed files with 11 additions and 23 deletions
@@ -13,7 +13,6 @@ import com.facebook.common.logging.FLog;
import com.facebook.infer.annotation.Assertions;
import com.facebook.react.bridge.ReadableArray;
import com.facebook.react.bridge.ReadableMapKeySetIterator;
import com.facebook.react.config.ReactFeatureFlags;
/**
* Class responsible for optimizing the native view hierarchy while still respecting the final UI
@@ -425,18 +424,16 @@ public class NativeViewHierarchyOptimizer {
// Bit of a hack: we need to update the layout of this node's children now that it's no longer
// layout-only, but we may still receive more layout updates at the end of this batch that we
// don't want to ignore.
if (ReactFeatureFlags.enableTransitionLayoutOnlyViewCleanup) {
FLog.i(
TAG,
"Transitioning LayoutOnlyView - tag: "
+ node.getReactTag()
+ " - rootTag: "
+ node.getRootTag()
+ " - hasProps: "
+ (props != null)
+ " - tagsWithLayout.size: "
+ mTagsWithLayoutVisited.size());
}
FLog.i(
TAG,
"Transitioning LayoutOnlyView - tag: "
+ node.getReactTag()
+ " - rootTag: "
+ node.getRootTag()
+ " - hasProps: "
+ (props != null)
+ " - tagsWithLayout.size: "
+ mTagsWithLayoutVisited.size());
Assertions.assertCondition(mTagsWithLayoutVisited.size() == 0);
applyLayoutBase(node);
for (int i = 0; i < node.getChildCount(); i++) {