v2.57.3 (#344)

* fix: drop libvrapi.so and deprecate OVR to fix Android 16KB launch crash (#491)

* fix: sample real LiDAR depthConfidence from ARKit confidenceMap on iOS

* Front-camera AR: use a registered provider instead of ARFaceTrackingConfiguration

* chore: update changelog

* fix: add missing files for camera provider

* fix: Crash on background/foreground transition with ViroVideo

* fix: ViroVideo teardown/lifecycle deadlocks on Android

* fix: anchor DepthPoint hit-test results without native ARHitTestResult (iOS)

* docs: changelog for ViroVideo #478 and DepthPoint anchor fixes

---------

Co-authored-by: Ernesto Mendoza <emhgerencia@gmail.com>

Author: doranteseduardo

CHANGELOG.md

@@ -1,5 +1,24 @@
 # CHANGELOG
 
+## v2.57.3 — 2 July 2026
+
+### Changed
+
+- **Front-camera AR no longer references the ARKit face-tracking / TrueDepth API in core (iOS).** `VROARSessioniOS` previously instantiated `ARFaceTrackingConfiguration` directly whenever `frontCameraEnabled` was set — so the TrueDepth symbol was compiled into every ViroKit binary and Apple's static App Store scan (Guideline 2.5.1) flagged *all* apps, even rear-camera-only ones that never touch the front camera. The front-camera path now consults a process-wide configuration provider (`VROARSessioniOS::setFrontCameraConfigProvider`) and runs whatever `ARConfiguration` it returns via `-runWithConfiguration:` (config was already stored/run as the base class, so nothing downstream changed); when no provider is registered it falls through to world tracking. A new Objective-C registration host, `VROFrontCameraProvider`, forwards to that setter and is discovered at runtime via `NSClassFromString` by the optional [`@reactvision/react-viro-face-tracking`](https://www.npmjs.com/package/@reactvision/react-viro-face-tracking) package, which is now the *only* place that references `ARFaceTrackingConfiguration`. Result: the core ViroKit binary is free of the TrueDepth API, and only apps that opt into front-camera face tracking carry (and declare) it.
+
+### Fixed
+
+- **ViroVideo Android lifecycle crash & ANR fixed (viro#478).** (1) `AVPlayer` guards its ExoPlayer listener callbacks with a `mDestroyed` flag set before release, so callbacks arriving after the native `VROAVPlayer` is deleted no longer reach freed memory (the background/foreground crash). (2) The GL render thread no longer blocks on the main thread calling ExoPlayer: `getCurrentTimeInSeconds()`/`getVideoDurationInSeconds()` read a main-thread-refreshed `volatile` cache, and `play()`/`pause()` are fire-and-forget — removing a deadlock against `GLSurfaceView.surfaceDestroyed()` that froze/ANR'd the app when leaving a ViroVideo screen (also per-frame playback jank). `nativeDeleteVideoTexture` no longer pauses on the GL thread before delete, and `destroy()` posts `ExoPlayer.release()` off the teardown call.
+- **AR hit-test results derived from depth data can now be anchored (iOS).** `createAnchoredNodeAtHitLocation()` returned null when a hit result had no underlying `ARHitTestResult` — e.g. a depth-synthesized `DepthPoint` — so `createAnchoredNode` failed with "hit result type does not support anchors". It now builds the `ARAnchor` from the result's own world transform in that case.
+- **iOS LiDAR `depthConfidence` now reports real confidence instead of the depth value.** In AR hit-test results the LiDAR confidence was sampled with `sampleDepthTextureAtUV()`, which always reads `ARFrame.sceneDepth.depthMap` regardless of the texture passed — so `depthConfidence` returned the depth (in metres) rather than a confidence. A dedicated `sampleConfidenceAtUV()` now reads ARKit's `sceneDepth.confidenceMap` (`ARConfidenceLevel`) and normalises it to `[0,1]` (low=0.0, medium=0.5, high=1.0; `-1.0` when unavailable). The existing LiDAR confidence gate (`> 0.3`) now works as intended, discarding low-confidence depth hits. The monocular path was unaffected.
+- **Android 15+ 16 KB launch crash — removed the non-compliant `libvrapi.so` (viro#491).** v2.57.2 aligned every `PT_LOAD` segment to ≥ 16 KB, but the prebuilt `libvrapi.so` (Meta VrApi / Oculus Mobile SDK) still had a `PT_GNU_RELRO` segment ending at `0x19000` — a 4 KB boundary, not a 16 KB one — leaving ~680 bytes of non-RELRO data sharing the tail page. Android 15+ rejects this (`program alignment (4096) cannot be smaller than system page size (16384)`; APK Analyzer: "RELRO is not a suffix and its end is not 16 KB aligned"). A field-level `p_align` patch on a stripped prebuilt cannot re-pad RELRO — only a real relink can — and `libviro_renderer.so` listed `libvrapi.so` as `NEEDED`, so it was force-loaded on **every** launch, crashing all apps (AR / GVR / Quest) on 16 KB-page devices.
+
+### Removed
+
+- **Deprecated and removed the Oculus Mobile SDK (VrApi) renderer path.** `VROSceneRendererOVR` / `VROInputControllerOVR` are no longer compiled, `lib-ovr` is unlinked, and `libvrapi.so` is dropped from `jniLibs` (both ABIs), so `libviro_renderer.so` no longer depends on it. VrApi targets EOL hardware (GearVR / Oculus Go); all current Meta headsets use the OpenXR path (`VROSceneRendererOpenXR`, added in 2.55.0–2.57.1). `nativeCreateRendererOVR` now returns 0 and `ViroViewOVR` is `@Deprecated`.
+
+---
+
 ## v2.57.2 — 29 June 2026
 
 ### Fixed

ViroRenderer/capi/VideoTexture_JNI.cpp

@@ -105,8 +105,11 @@ VRO_METHOD(void, nativeAttachDelegate)(VRO_ARGS
 
 VRO_METHOD(void, nativeDeleteVideoTexture)(VRO_ARGS
                                            VRO_REF(VROVideoTexture) textureRef) {
-    std::shared_ptr<VROVideoTexture> videoTexture = VRO_REF_GET(VROVideoTexture, textureRef);
-    videoTexture->pause();
+    // Do NOT pause() here. This can run on the GL/render thread during scene teardown,
+    // and pause() bounces to the main thread (ExoPlayer must be touched there) — which is
+    // simultaneously blocked in GLSurfaceView.surfaceDestroyed() waiting for this GL thread,
+    // deadlocking (ANR on exit). Deleting the texture already tears the player down:
+    // ~VROVideoTextureAVP -> delete _player -> ~VROAVPlayer -> AVPlayer.destroy() (stop+release).
     VRO_REF_DELETE(VROVideoTexture, textureRef);
 }
 

android/sharedCode/CMakeLists.txt

@@ -54,7 +54,9 @@ add_library( # Sets the name of the library.
              ${VIRO_ANDROID_SRC}/VROLooper.cpp
              ${VIRO_ANDROID_SRC}/VROSceneRenderer.cpp
              ${VIRO_ANDROID_SRC}/VROSceneRendererGVR.cpp
-             ${VIRO_ANDROID_SRC}/VROSceneRendererOVR.cpp
+             # VROSceneRendererOVR.cpp (VrApi / GearVR-Oculus Go) deprecated in 2.57.3 —
+             # superseded by the OpenXR (Quest) path. Excluding it drops the libvrapi.so
+             # dependency, which was not 16 KB-page compliant (see viro#491).
              ${VIRO_ANDROID_SRC}/VROSceneRendererOpenXR.cpp
              ${VIRO_ANDROID_SRC}/VROARSessionOpenXR.cpp
              ${VIRO_ANDROID_SRC}/VROSceneRendererSceneView.cpp
@@ -68,7 +70,7 @@ add_library( # Sets the name of the library.
              ${VIRO_ANDROID_SRC}/VROTypefaceAndroid.cpp
              ${VIRO_ANDROID_SRC}/VROInputControllerDaydream.cpp
              ${VIRO_ANDROID_SRC}/VROInputControllerCardboard.cpp
-             ${VIRO_ANDROID_SRC}/VROInputControllerOVR.cpp
+             # VROInputControllerOVR.cpp deprecated in 2.57.3 (VrApi path removed).
              ${VIRO_ANDROID_SRC}/VROInputControllerOpenXR.cpp
              ${VIRO_ANDROID_SRC}/VROAVRecorderAndroid.cpp
              ${VIRO_ANDROID_SRC}/VROTextureReader.cpp
@@ -399,12 +401,11 @@ set_target_properties( lib-gvr-audio
                        PROPERTIES IMPORTED_LOCATION
                        ${VIRO_ANDROID_LIBS}/${ANDROID_ABI}/libgvr_audio.so )
 
-add_library( lib-ovr
-             SHARED
-             IMPORTED )
-set_target_properties( lib-ovr
-                       PROPERTIES IMPORTED_LOCATION
-                       ${VIRO_ANDROID_LIBS}/${ANDROID_ABI}/libvrapi.so )
+# lib-ovr (libvrapi.so — Meta VrApi / Oculus Mobile SDK) removed in 2.57.3.
+# VrApi targets EOL hardware (GearVR / Oculus Go); all current Meta headsets use
+# the OpenXR path (libopenxr_loader.so). The prebuilt libvrapi.so was not 16 KB
+# page-size compliant (PT_GNU_RELRO not 16 KB-aligned) and crashed Android 15+
+# 16 KB devices on launch because libviro_renderer.so listed it as NEEDED (viro#491).
 
 add_library( lib-freetype
              SHARED
@@ -509,7 +510,6 @@ target_link_libraries( # Specifies the target library.
                        atomic
                        lib-gvr
                        lib-gvr-audio
-                       lib-ovr
                        lib-freetype
                        lib-protobuf
                        lib-BulletCollision

android/sharedCode/src/main/cpp/jni/VRORenderer_JNI.cpp

@@ -33,7 +33,7 @@
 #include "vr/gvr/capi/include/gvr_audio.h"
 #include "VROProjector.h"
 #include "VROSceneRendererGVR.h"
-#include "VROSceneRendererOVR.h"
+// VROSceneRendererOVR.h (VrApi) removed in 2.57.3 — see nativeCreateRendererOVR below.
 #include "VROSceneRendererOpenXR.h"
 #include "VROSceneRendererSceneView.h"
 #include "VROPlatformUtil.h"
@@ -102,21 +102,13 @@ VRO_METHOD(jlong, nativeCreateRendererOVR)(VRO_ARGS
                                            jboolean enableHDR,
                                            jboolean enablePBR,
                                            jboolean enableBloom) {
-    VROPlatformSetType(VROPlatformType::AndroidOVR);
-
-    std::shared_ptr<gvr::AudioApi> gvrAudio = std::make_shared<gvr::AudioApi>();
-    gvrAudio->Init(env, android_context, class_loader, GVR_AUDIO_RENDERING_BINAURAL_HIGH_QUALITY);
-    VROPlatformSetEnv(env, android_context, asset_mgr, platform_util);
-
-    VRORendererConfiguration config;
-    config.enableShadows = enableShadows;
-    config.enableHDR = enableHDR;
-    config.enablePBR = enablePBR;
-    config.enableBloom = enableBloom;
-
-    std::shared_ptr<VROSceneRenderer> renderer
-            = std::make_shared<VROSceneRendererOVR>(config, gvrAudio, view, activity, env);
-    return Renderer::jptr(renderer);
+    // Deprecated in 2.57.3. The VrApi (Oculus Mobile SDK / GearVR / Oculus Go)
+    // renderer was removed together with libvrapi.so, which was not 16 KB
+    // page-size compliant and crashed Android 15+ 16 KB devices (viro#491).
+    // All supported Meta headsets use the OpenXR path (nativeCreateRendererOpenXR).
+    // Returns 0 so the Java layer degrades gracefully instead of loading VrApi.
+    pinfo("ViroViewOVR / OVR_MOBILE is deprecated (removed in 2.57.3) — use the QUEST (OpenXR) platform instead");
+    return 0;
 }
 
 VRO_METHOD(jlong, nativeCreateRendererOpenXR)(VRO_ARGS
@@ -453,12 +445,14 @@ VRO_METHOD(void, nativeSetDebugHUDEnabled)(VRO_ARGS
     renderer->getRenderer()->setDebugHUDEnabled(enabled);
 }
 
-// This function is OVR only!
+// VR-only (OpenXR / Quest). The legacy OVR (VrApi) path was removed in 2.57.3.
 VRO_METHOD(void, nativeRecenterTracking)(VRO_ARGS
                                          jlong native_renderer) {
     std::shared_ptr<VROSceneRenderer> renderer = Renderer::native(native_renderer);
-    std::shared_ptr<VROSceneRendererOVR> ovrRenderer = std::dynamic_pointer_cast<VROSceneRendererOVR>(renderer);
-    ovrRenderer->recenterTracking();
+    std::shared_ptr<VROSceneRendererOpenXR> xrRenderer = std::dynamic_pointer_cast<VROSceneRendererOpenXR>(renderer);
+    if (xrRenderer) {
+        xrRenderer->recenterTracking();
+    }
 }
 
 VRO_METHOD(VRO_FLOAT_ARRAY, nativeProjectPoint)(VRO_ARGS

android/sharedCode/src/main/java/com/viro/core/Renderer.java

@@ -96,7 +96,8 @@ public void drawFrame() {
     }
     public void setVRModeEnabled(boolean enabled) { nativeSetVRModeEnabled(mNativeRef, enabled); }
 
-    /* ----------     OVR only methods    ---------- */
+    /* ----------     OVR only methods (deprecated 2.57.3 — VrApi removed, see viro#491)    ---------- */
+    @Deprecated
     public Renderer(ClassLoader appClassLoader, Context context,
                     ViroViewOVR view, Activity activity, AssetManager assets, PlatformUtil platformUtil,
                     RendererConfiguration config) {

android/sharedCode/src/main/java/com/viro/core/VideoTexture.java

@@ -190,15 +190,19 @@ public void dispose() {
      */
     public void play() {
         mPaused = false;
-        nativePlay(mNativeRef);
+        if (mNativeRef != 0) {
+            nativePlay(mNativeRef);
+        }
     }
 
     /**
      * Pause the video.
      */
     public void pause() {
         mPaused = true;
-        nativePause(mNativeRef);
+        if (mNativeRef != 0) {
+            nativePause(mNativeRef);
+        }
     }
 
     /**

android/sharedCode/src/main/java/com/viro/core/ViroViewOVR.java

@@ -52,7 +52,14 @@
 /**
  * ViroViewOVR is a {@link ViroView} for rendering content in stereo for VR headsets using
  * the Oculus Mobile SDK.
+ *
+ * @deprecated Deprecated in 2.57.3. The Oculus Mobile SDK (VrApi) targets EOL hardware
+ * (GearVR / Oculus Go) and its native library ({@code libvrapi.so}) is not 16 KB page-size
+ * compliant, crashing Android 15+ 16 KB devices on launch (viro#491). Use {@link ViroViewOpenXR}
+ * with the {@code QUEST} platform for all current Meta headsets. This class no longer creates a
+ * native renderer.
  */
+@Deprecated
 public class ViroViewOVR extends ViroView implements SurfaceHolder.Callback {
 
     static {