From 8d0c3285edf0e805077ced3382de15817e76fe02 Mon Sep 17 00:00:00 2001 From: Nathan Walker Date: Sat, 2 Jul 2022 10:07:04 -0700 Subject: [PATCH] chore: cleanup uifont cache logs to be trace enabled --- packages/core/ui/styling/font.ios.ts | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/packages/core/ui/styling/font.ios.ts b/packages/core/ui/styling/font.ios.ts index 23afc9fd7..aa7535804 100644 --- a/packages/core/ui/styling/font.ios.ts +++ b/packages/core/ui/styling/font.ios.ts @@ -13,9 +13,6 @@ interface FontDescriptor { const uiFontCache = new Map(); -let hitCount = 0; -let missCount = 0; - function computeFontCacheKey(fontDescriptor: FontDescriptor) { const { fontFamily, fontSize, fontWeight, isBold, isItalic } = fontDescriptor; const sep = ':'; @@ -26,12 +23,16 @@ function getUIFontCached(fontDescriptor: FontDescriptor) { const cacheKey = computeFontCacheKey(fontDescriptor); if (uiFontCache.has(cacheKey)) { - console.log('uiFont cache hit', ++hitCount, '| misses:', missCount); + if (Trace.isEnabled()) { + Trace.write(`UIFont reuse from cache: ${JSON.stringify(fontDescriptor)}, cache size: ${uiFontCache.size}`, Trace.categories.Style, Trace.messageType.info); + } return uiFontCache.get(cacheKey); } - console.log('uiFont cache miss', ++missCount); const uiFont = NativeScriptUtils.createUIFont(fontDescriptor as any); uiFontCache.set(cacheKey, uiFont); + if (Trace.isEnabled()) { + Trace.write(`UIFont creation: ${JSON.stringify(fontDescriptor)}, cache size: ${uiFontCache.size}`, Trace.categories.Style, Trace.messageType.info); + } return uiFont; }