From ec27141b93cf94c14762399c45ec0d25366afe9a Mon Sep 17 00:00:00 2001 From: David Vacca Date: Tue, 1 Mar 2022 14:56:44 -0800 Subject: [PATCH] remove duplicate code in FabricUIManager Summary: remove duplicate code in FabricUIManager The removed line is already called few lines above changelog: [internal] internal Reviewed By: JoshuaGross Differential Revision: D34553849 fbshipit-source-id: 2e0102a77e7464c9803fb1175385fdafbb88f1fa --- .../main/java/com/facebook/react/fabric/FabricUIManager.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricUIManager.java b/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricUIManager.java index b14285c1a0e..9291b5e335a 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricUIManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricUIManager.java @@ -428,11 +428,6 @@ public class FabricUIManager implements UIManager, LifecycleEventListener { mReactApplicationContext.removeLifecycleEventListener(this); onHostPause(); - // This is not technically thread-safe, since it's read on the UI thread and written - // here on the JS thread. We've marked it as volatile so that this writes to UI-thread - // memory immediately. - mDispatchUIFrameCallback.stop(); - mBinding.unregister(); mBinding = null;