Skip to content

Commit 49267d4

Browse files
cortinicometa-codesync[bot]
authored andcommitted
Cleanup unnecessary logging on ReactHostImpl
Summary: Those stacktraces are no longer necessary because venice has been rolled out to 100% since a long time. They create noise on logcat as they appear as a crash in red, but they're not. Created from CodeHub with https://fburl.com/edit-in-codehub Changelog: [Internal] [Changed] - Reviewed By: javache Differential Revision: D88075839 fbshipit-source-id: 83f2c5fe6b3e500c5a0dc12a81ded394b5fca31a
1 parent e167faa commit 49267d4

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

  • packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/ReactHostImpl.kt

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1266,10 +1266,6 @@ public class ReactHostImpl(
12661266
val method = "getOrCreateReloadTask()"
12671267
stateTracker.enterState(method)
12681268

1269-
// Log how React Native is destroyed
1270-
// TODO(T136397487): Remove after Venice is shipped to 100%
1271-
raiseSoftException(method, reason)
1272-
12731269
reloadTask?.let {
12741270
return it
12751271
}
@@ -1421,10 +1417,6 @@ public class ReactHostImpl(
14211417
val method = "getOrCreateDestroyTask()"
14221418
stateTracker.enterState(method)
14231419

1424-
// Log how React Native is destroyed
1425-
// TODO(T136397487): Remove after Venice is shipped to 100%
1426-
raiseSoftException(method, reason, ex)
1427-
14281420
destroyTask?.let {
14291421
return it
14301422
}

0 commit comments

Comments
 (0)