From 9380ec0d23cc491f10bbabf75488559a6ee4a9bb Mon Sep 17 00:00:00 2001
From: David Vacca <dvacca@fb.com>
Date: Fri, 4 Jan 2019 16:33:39 -0800
Subject: [PATCH] Fix 'global reference table overflow' error in Fabric

Summary: This diff ensures that the 'measure' method in TextLayoutManager is memoized using a static variable.

Reviewed By: fkgozali

Differential Revision: D13585508

fbshipit-source-id: 9275a4d193b8abb0c3aaffd5a5535234717475e1
---
 .../platform/android/TextLayoutManager.cpp    | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/ReactCommon/fabric/textlayoutmanager/platform/android/TextLayoutManager.cpp b/ReactCommon/fabric/textlayoutmanager/platform/android/TextLayoutManager.cpp
index c39dbb769b1..35cb5e43573 100644
--- a/ReactCommon/fabric/textlayoutmanager/platform/android/TextLayoutManager.cpp
+++ b/ReactCommon/fabric/textlayoutmanager/platform/android/TextLayoutManager.cpp
@@ -30,16 +30,16 @@ Size TextLayoutManager::measure(
       contextContainer_->getInstance<jni::global_ref<jobject>>(
           "FabricUIManager");
 
-  auto clazz =
-      jni::findClassStatic("com/facebook/fbreact/fabric/FabricUIManager");
-  static auto measure = clazz->getMethod<jlong(
-      jstring,
-      ReadableNativeMap::javaobject,
-      ReadableNativeMap::javaobject,
-      jint,
-      jint,
-      jint,
-      jint)>("measure");
+  static auto measure =
+      jni::findClassStatic("com/facebook/fbreact/fabric/FabricUIManager")
+          ->getMethod<jlong(
+              jstring,
+              ReadableNativeMap::javaobject,
+              ReadableNativeMap::javaobject,
+              jint,
+              jint,
+              jint,
+              jint)>("measure");
 
   auto minimumSize = layoutConstraints.minimumSize;
   auto maximumSize = layoutConstraints.maximumSize;