From 38062cf8a7e17bbfa976f1b9332706d7cd032876 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rub=C3=A9n=20Norte?= <rubennorte@fb.com>
Date: Thu, 3 Sep 2020 06:43:50 -0700
Subject: [PATCH] Log renderApplication_React_render timespan to scoped
 performance logger if available

Summary: Changelog: [Changed] Used scoped performance logger to mark the beginning and end of the call to the renderer, instead of using `GlobalPerformanceLogger`.

Reviewed By: lunaleaps

Differential Revision: D23240784

fbshipit-source-id: 7099f1a4b5da3f29596fcee289a715d295ae0a65
---
 Libraries/ReactNative/renderApplication.js | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/Libraries/ReactNative/renderApplication.js b/Libraries/ReactNative/renderApplication.js
index 7d7b3b4409b..a757414252d 100644
--- a/Libraries/ReactNative/renderApplication.js
+++ b/Libraries/ReactNative/renderApplication.js
@@ -33,9 +33,10 @@ function renderApplication<Props: Object>(
 ) {
   invariant(rootTag, 'Expect to have a valid rootTag, instead got ', rootTag);
 
+  const performanceLogger = scopedPerformanceLogger ?? GlobalPerformanceLogger;
+
   const renderable = (
-    <PerformanceLoggerContext.Provider
-      value={scopedPerformanceLogger ?? GlobalPerformanceLogger}>
+    <PerformanceLoggerContext.Provider value={performanceLogger}>
       <AppContainer
         rootTag={rootTag}
         fabric={fabric}
@@ -48,13 +49,13 @@ function renderApplication<Props: Object>(
     </PerformanceLoggerContext.Provider>
   );
 
-  GlobalPerformanceLogger.startTimespan('renderApplication_React_render');
+  performanceLogger.startTimespan('renderApplication_React_render');
   if (fabric) {
     require('../Renderer/shims/ReactFabric').render(renderable, rootTag);
   } else {
     require('../Renderer/shims/ReactNative').render(renderable, rootTag);
   }
-  GlobalPerformanceLogger.stopTimespan('renderApplication_React_render');
+  performanceLogger.stopTimespan('renderApplication_React_render');
 }
 
 module.exports = renderApplication;