Do not return released component trees in cached values

Summary: Do not return released component trees in cached values

Reviewed By: fbcbl

Differential Revision: D67027304

fbshipit-source-id: fe64f5c9f6c0080146ca003ad1cf9399ff1e34f4

litho-core/src/main/java/com/facebook/litho/ComponentTree.java

@@ -2532,7 +2532,19 @@ public void release() {
     if (mReleased || mTreeState == null) {
       return null;
     }
-    return mTreeState.getCachedValue(globalKey, index, cachedValueInputs, isLayoutState);
+    Object cachedValue =
+        mTreeState.getCachedValue(globalKey, index, cachedValueInputs, isLayoutState);
+    if (ComponentsConfiguration.defaultInstance.enableVisibilityFixForNestedLithoView) {
+      // This is a temporary solution to solve the issue of visibility events for
+      // VerticalScrollComponentSpec and HorizontalScrollSpec. We can remove it after shipping the
+      // primitive version of VerticalScroll and HorizontalScroll
+      if (cachedValue instanceof ComponentTree) {
+        if (((ComponentTree) cachedValue).isReleased()) {
+          return null;
+        }
+      }
+    }
+    return cachedValue;
   }
 
   @VisibleForTesting