Fix Systrace section for Lazy View Managers

Differential Revision: D6088310

fbshipit-source-id: cf14c9cc91068cfabf4e7c1bbae5df7f25e0ced3
This commit is contained in:
Alexey Lang 2017-10-18 12:13:06 -07:00 committed by Facebook Github Bot
parent f0ac5b369a
commit 52a67db629
2 changed files with 5 additions and 4 deletions

View File

@ -39,7 +39,6 @@ import com.facebook.react.uimanager.debug.NotThreadSafeViewHierarchyUpdateDebugL
import com.facebook.react.uimanager.events.EventDispatcher;
import com.facebook.systrace.Systrace;
import com.facebook.systrace.SystraceMessage;
import java.util.ArrayList;
import java.util.List;
import java.util.Map;
@ -245,7 +244,8 @@ public class UIManagerModule extends ReactContextBaseJavaModule implements
return null;
}
SystraceMessage.beginSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "constants for ViewManager")
SystraceMessage.beginSection(
Systrace.TRACE_TAG_REACT_JAVA_BRIDGE, "UIManagerModule.getConstantsForViewManager")
.arg("ViewManager", targetView.getName())
.arg("Lazy", true)
.flush();
@ -263,7 +263,7 @@ public class UIManagerModule extends ReactContextBaseJavaModule implements
}
return null;
} finally {
SystraceMessage.endSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE);
SystraceMessage.endSection(Systrace.TRACE_TAG_REACT_JAVA_BRIDGE).flush();
}
}

View File

@ -75,7 +75,8 @@ import javax.annotation.Nullable;
for (ViewManager viewManager : viewManagers) {
final String viewManagerName = viewManager.getName();
SystraceMessage.beginSection(TRACE_TAG_REACT_JAVA_BRIDGE, "constants for ViewManager")
SystraceMessage.beginSection(
TRACE_TAG_REACT_JAVA_BRIDGE, "UIManagerModuleConstantsHelper.createConstants")
.arg("ViewManager", viewManagerName)
.arg("Lazy", false)
.flush();