From 164e133a98fade1c0f7577476e411bf9f6c2b7ca Mon Sep 17 00:00:00 2001 From: David Vacca Date: Mon, 25 Jan 2021 03:02:07 -0800 Subject: [PATCH] Remove usage of FabricUIManager Module related to NativeDriverAnimations Summary: This diff removes references to FabricUIManager from UIManagerModule, these callsited were originally used for NativeAnimatedDriver, but they are not used anymore changelog: [internal] internal Reviewed By: JoshuaGross, shergin Differential Revision: D26035388 fbshipit-source-id: d4825af17f6948d922c42670f2c7b02498c12039 --- .../react/uimanager/UIManagerModule.java | 31 ++----------------- 1 file changed, 2 insertions(+), 29 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java index 9ef007a6587..36457847cb1 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java @@ -421,16 +421,7 @@ public class UIManagerModule extends ReactContextBaseJavaModule */ @Override public void synchronouslyUpdateViewOnUIThread(int tag, ReadableMap props) { - int uiManagerType = ViewUtil.getUIManagerType(tag); - if (uiManagerType == FABRIC) { - UIManager fabricUIManager = - UIManagerHelper.getUIManager(getReactApplicationContext(), uiManagerType); - if (fabricUIManager != null) { - fabricUIManager.synchronouslyUpdateViewOnUIThread(tag, props); - } - } else { - mUIImplementation.synchronouslyUpdateViewOnUIThread(tag, new ReactStylesDiffMap(props)); - } + mUIImplementation.synchronouslyUpdateViewOnUIThread(tag, new ReactStylesDiffMap(props)); } /** @@ -533,25 +524,7 @@ public class UIManagerModule extends ReactContextBaseJavaModule FLog.d(ReactConstants.TAG, message); PrinterHolder.getPrinter().logMessage(ReactDebugOverlayTags.UI_MANAGER, message); } - int uiManagerType = ViewUtil.getUIManagerType(tag); - if (uiManagerType == FABRIC) { - ReactApplicationContext reactApplicationContext = getReactApplicationContext(); - if (reactApplicationContext.hasActiveCatalystInstance()) { - final UIManager fabricUIManager = - UIManagerHelper.getUIManager(reactApplicationContext, uiManagerType); - if (fabricUIManager != null) { - reactApplicationContext.runOnUiQueueThread( - new Runnable() { - @Override - public void run() { - fabricUIManager.synchronouslyUpdateViewOnUIThread(tag, props); - } - }); - } - } - } else { - mUIImplementation.updateView(tag, className, props); - } + mUIImplementation.updateView(tag, className, props); } /**