Call cleanup of insertion effects when hidden (#30954)

Summary:
bypass
Insertion effects do not unmount when a subtree is removed while
offscreen.

Current behavior for an insertion effect is if the component goes

- *visible -> removed:* calls insertion effect cleanup
- *visible -> offscreen -> removed:* insertion effect cleanup is never
called

This makes it so we always call insertion effect cleanup when removing
the component.

Likely also fixes facebook/react#26670

---------

DiffTrain build for commit facebook/react@d3d4d3a.

Reviewed By: kassens

Differential Revision: D62661120

Pulled By: josephsavona

fbshipit-source-id: 81412821e4f100fb5acb4b54e8946fe03b389115

Co-authored-by: Rick Hanlon <rickhanlonii@fb.com>

packages/react-relay/relay-hooks/__tests__/useQueryLoader-test.js

@@ -755,11 +755,7 @@ describe.each([
     expect(outerInstance?.toJSON()).toEqual('fallback');
     expect(releaseQuery).not.toHaveBeenCalled();
     ReactTestRenderer.act(() => outerInstance?.unmount());
-    if (ENABLE_ACTIVITY_COMPATIBILITY) {
-      expect(releaseQuery).toHaveBeenCalledTimes(0);
-      jest.runAllTimers();
-      expect(releaseQuery).toHaveBeenCalledTimes(1);
-    } else {
+    if (!ENABLE_ACTIVITY_COMPATIBILITY) {
       expect(releaseQuery).toHaveBeenCalledTimes(1);
     }
   });
@@ -812,11 +808,7 @@ describe.each([
     expect(outerInstance?.toJSON()).toEqual('fallback');
     expect(releaseQuery).not.toHaveBeenCalled();
     ReactTestRenderer.act(() => outerInstance?.unmount());
-    if (ENABLE_ACTIVITY_COMPATIBILITY) {
-      expect(releaseQuery).toHaveBeenCalledTimes(0);
-      jest.runAllTimers();
-      expect(releaseQuery).toHaveBeenCalledTimes(1);
-    } else {
+    if (!ENABLE_ACTIVITY_COMPATIBILITY) {
       expect(releaseQuery).toHaveBeenCalledTimes(1);
     }
   });