react-native-video works in debug but not release builds

[ryfow]

react-native-video works in debug but not release builds

Environment

PS C:\Projects\RNVProblem> npx react-native-version
npx: installed 51 in 5.957s
[RNV] Versioning Android...
[RNV] Android updated
[RNV] Versioning iOS...
[RNV] iOS updated
[RNV] Done
PS C:\Projects\RNVProblem> npx react-native info
info Fetching system and libraries information...
System:
    OS: Windows 10 10.0.19041
    CPU: (10) x64 Intel(R) Xeon(R) Gold 6126 CPU @ 2.60GHz
    Memory: 6.18 GB / 12.00 GB
  Binaries:
    Node: 14.8.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.4 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.7 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK: Not Found
    Windows SDK:
      AllowDevelopmentWithoutDevLicense: Enabled
      AllowAllTrustedApps: Enabled
      Versions: 10.0.15063.0, 10.0.16299.0, 10.0.18362.0, 10.0.19041.0
  IDEs:
    Android Studio: Not Found
    Visual Studio: 16.9.31229.75 (Visual Studio Professional 2019), 16.7.31229.181 (Visual Studio Enterprise 2019)
  Languages:
    Java: 11.0.8 - C:\Program Files\Zulu\zulu-11\bin\javac.EXE
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.1 => 17.0.1
    react-native: 0.64.0 => 0.64.0
    react-native-windows: ^0.64.0-0 => 0.64.9
  npmGlobalPackages:
    *react-native*: Not Found
PS C:\Projects\RNVProblem>

Steps To Reproduce

Provide a detailed list of steps that reproduce the issue.

1. Try running a release build of https://github.com/ryfow/RNVProblem on Windows

Expected Results

I expect a screen to render and some audio to play when the app runs.

Instead, I see a white screen, hear no audio, and see this exception in the debugger when all C++ exceptions break.

Snack, code example, screenshot, or link to a repository:

https://github.com/ryfow/RNVProblem

[asklar]

could be because the module is accessing ui stuff from the js thread?
cc @vmoroz

[chrisglein]

Error appears to be in JavaScript, so the native stack isn't as useful.

@jonthysell are there any known/unpublished issues with react-native-video here?

[chrisglein]

@ryfow Can you try running this in debug but while disabling web debugging? One hypothesis is that's the difference here, and you'd get more context in debug.

[ryfow]

I get this when the app reloads after turning debug off. Is there a way to start with debug off?

[acoates-ms]

Yup, this is a bug in react-native-video.

https://github.com/react-native-video/react-native-video/blob/1eb11ce7b7619a04e65affedcb1ffff4f82f0af3/Video.js#L257
They should be importing UIManager from react-native, not accessing it from NativeModules.UIManager.

It looks like there is already a PR open to do this:
TheWidlarzGroup/react-native-video#2296

[ryfow]

That PR commit appears to have the same problem in Release build and gives me a new error in debug mode, slightly different depending on whether remote debugging in on.

Debugging on:

Debugging off:

[ryfow]

Oh, the other thing I noticed when staring at the VS debugger yesterday is that the call stack shows things are blowing up under JSIExecutor::callFunction where moduleId = "RCTEventEmitter" and methodId = "receiveTouches".

[KarkaLT]

I have exactly the same issue, have you found a workaround?