From 10b521815c1b5a73bf0e27b85e7286be63f78f13 Mon Sep 17 00:00:00 2001 From: David Vacca Date: Mon, 28 Jan 2019 12:56:56 -0800 Subject: [PATCH] Add QPL marker to track time it takes to load .so file from RN Summary: Simple diff that adds QPL marker to track time it takes to load .so file from RN Reviewed By: fkgozali Differential Revision: D13845379 fbshipit-source-id: 6c2272605ba3fb08ab8ebce867f84c6a488438d0 --- .../src/main/java/com/facebook/react/bridge/ReactBridge.java | 2 ++ .../java/com/facebook/react/bridge/ReactMarkerConstants.java | 2 ++ 2 files changed, 4 insertions(+) diff --git a/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactBridge.java b/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactBridge.java index c019830eb69..db7edacb75a 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactBridge.java +++ b/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactBridge.java @@ -27,7 +27,9 @@ public class ReactBridge { sLoadStartTime = SystemClock.uptimeMillis(); Systrace.beginSection(TRACE_TAG_REACT_JAVA_BRIDGE, "ReactBridge.staticInit::load:reactnativejni"); + ReactMarker.logMarker(ReactMarkerConstants.LOAD_REACT_NATIVE_SO_FILE_START); SoLoader.loadLibrary("reactnativejni"); + ReactMarker.logMarker(ReactMarkerConstants.LOAD_REACT_NATIVE_SO_FILE_END); Systrace.endSection(TRACE_TAG_REACT_JAVA_BRIDGE); sLoadEndTime = SystemClock.uptimeMillis(); } diff --git a/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactMarkerConstants.java b/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactMarkerConstants.java index 270b0a6ab4c..4ba2791aa1e 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactMarkerConstants.java +++ b/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactMarkerConstants.java @@ -92,4 +92,6 @@ public enum ReactMarkerConstants { ON_FRAGMENT_CREATE, JAVASCRIPT_EXECUTOR_FACTORY_INJECT_START, JAVASCRIPT_EXECUTOR_FACTORY_INJECT_END, + LOAD_REACT_NATIVE_SO_FILE_START, + LOAD_REACT_NATIVE_SO_FILE_END, }