From f38ab24a192a9c2cd0a849fb07d0c7257feb3390 Mon Sep 17 00:00:00 2001 From: "Andrew Chen (Eng)" Date: Thu, 6 Sep 2018 20:56:37 -0700 Subject: [PATCH] Fix setupReactContext systrace Summary: setupReactContext wasn't ending because D9664719 added incorrect end markers. Need to use either Systrace.endSection() or SystraceMessage.endSection().flush(). Differential Revision: D9695854 fbshipit-source-id: e0f7d492bc58536ce242962438edac7eec671867 --- .../java/com/facebook/react/uimanager/UIManagerModule.java | 4 ++-- 1 file changed, 2 insertions(+), 2 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 37255c996..fcc11c6ed 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/UIManagerModule.java @@ -219,7 +219,7 @@ public class UIManagerModule extends ReactContextBaseJavaModule implements try { return UIManagerModuleConstantsHelper.createConstants(viewManagerResolver); } finally { - SystraceMessage.endSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE); + Systrace.endSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE); ReactMarker.logMarker(CREATE_UI_MANAGER_MODULE_CONSTANTS_END); } } @@ -237,7 +237,7 @@ public class UIManagerModule extends ReactContextBaseJavaModule implements return UIManagerModuleConstantsHelper.createConstants( viewManagers, customBubblingEvents, customDirectEvents); } finally { - SystraceMessage.endSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE); + Systrace.endSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE); ReactMarker.logMarker(CREATE_UI_MANAGER_MODULE_CONSTANTS_END); } }