Facebook/react-native
1
0
Fork 0
mirror of https://github.com/facebook/react-native.git synced 2025-11-01 09:14:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

Move Hermes code coverage to non-OSS (#36800)

Browse Source 
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/36800

Hermes code coverage was added in D43458602, move the logic to internal to avoid internal dependencies in Bridgeless.

Changelog:
[Android][Changed] - Move Hermes code coverage to non-OSS

Reviewed By: cortinico

Differential Revision: D44626679

fbshipit-source-id: 95d71ec6fe6c75664682511efb571a8122d955c2
This commit is contained in:
Lulu Wu
2023-04-19 13:15:47 -07:00
committed by Facebook GitHub Bot
parent 5763594cda
commit 93851fc2c8
1 changed files with 1 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridgeless/hermes/HermesInstance.kt
+1 -10
View File
@@ -7,16 +7,13 @@
package com.facebook.react.bridgeless.hermes
import com.facebook.common.process.ProcessName
import com.facebook.jni.HybridData
import com.facebook.jni.annotations.DoNotStrip
import com.facebook.jsi.mdcd.HermesCodeCoverage
import com.facebook.react.bridge.ReactApplicationContext
import com.facebook.react.bridgeless.JSEngineInstance
import com.facebook.soloader.SoLoader
import kotlin.jvm.JvmStatic
class HermesInstance(context: ReactApplicationContext) : JSEngineInstance(initHybrid()!!) {
class HermesInstance() : JSEngineInstance(initHybrid()!!) {
companion object {
@JvmStatic @DoNotStrip protected external fun initHybrid(): HybridData?
@@ -25,10 +22,4 @@ class HermesInstance(context: ReactApplicationContext) : JSEngineInstance(initHy
SoLoader.loadLibrary("hermesinstancejni")
}
}
init {
// Initialize Code Coverage tracing for select usecases, like E2E coverage and Hermes MDCD
val processName = ProcessName.current().fullName
HermesCodeCoverage.initialize(context, processName.orEmpty())
}
}