Error in reportMount when surface is not found (#46678)

Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/46678

I'm investigating some issues with surface management and noticed `surfaceHandlerRegistry_` is not updated in the bridgeless path when using ReactSurfaceView.

Adding a warning to help us validate this is resolved when we do eventually fix it.

Changelog: [Internal]

Reviewed By: fabriziocucci

Differential Revision: D63463521

fbshipit-source-id: 38995924588f1d71b9fc517c76a6e0c572fd0699
This commit is contained in:
Pieter De Baets
2024-09-26 11:32:00 -07:00
committed by Facebook GitHub Bot
parent 633ad4933e
commit 04da3d784b
@@ -109,11 +109,13 @@ void Binding::reportMount(SurfaceId surfaceId) {
// incorrectly. This is due to the push model used on Android and can be
// removed when we migrate to a pull model.
std::shared_lock lock(surfaceHandlerRegistryMutex_);
auto iterator = surfaceHandlerRegistry_.find(surfaceId);
if (iterator != surfaceHandlerRegistry_.end()) {
auto& surfaceHandler = iterator->second;
surfaceHandler.getMountingCoordinator()->didPerformAsyncTransactions();
} else {
LOG(ERROR) << "Binding::reportMount: Surface with id " << surfaceId
<< " is not found";
}
}