Android issues

[CedricGuillemet]

Ordered by complexity

• the project compiles for x86 and x86_64 but has never been tested on the simulator
• V8 .so are poorly handled in the APK for the Native Activity project. It needs to be fixed before it can run
• V8 must be installed manually for local builds. This should be done automatically in CMake
• Screen ratio is not updated with screen rotation
• Add support for SSL
• Shader compliance and parsing between BJS and bgfx is not great for OpenGL. Expect issues when dealing with more complex shaders
• Front facing is inverted (you see the inside of the cube). A bgfx repo update will be done shortly and should be done on Master @CedricGuillemet
• Asset loading is using a callback mecanism that supports APK assets loading. It will be replaced by a plugin extension (Investigate pluggable pattern for extending the runtime #101)
• getHostDocument is missing in native engine definition ( https://github.com/BabylonJS/BabylonNative/pull/30/files#diff-61eebfcda2bd45c2180b579c8f4fcd42R51 ) => PR : Native Babylon.js#7286

[CedricGuillemet]

the project compiles for x86 and x86_64 but has never been tested on the simulator

• Follow the readme to build for your device
• Add a virtual device in AVD (any Android 5.0+), build for the target, run in simulator.
• The cube should appear with touch support to rotate the cube.
• Test with other virtual devices and various Android version

DONE!

[CedricGuillemet]

V8 .so are poorly handled in the APK for the Native Activity project. It needs to be fixed before it can run

• Install V8 for Android and AndroidNativeActivity
• make sur Android project builds and runs on your device
• Open AndroidNativeActivity with Android Studio.
• Build for your device
• Run app
• check the logcat, you should see:

FATAL EXCEPTION: main
Unable to load native library: /data/app/com.android.testapp-2/lib/arm64/libnative-activity.so

Apparently, it's a runtime library dependency issue. With a sample from Google, I get the error when linking with V8. I think a .so is missing/not found.
AndroidNative Activity removed for now. Will add it back if needed.

[CedricGuillemet]

V8 must be installed manually for local builds. This should be done automatically in CMake

TestApp Android uses V8. I tried to compile an Android version a while ago without success. But found this working npm : https://www.npmjs.com/package/react-native-v8
The CI download and installs the .so where needed ( https://github.com/BabylonJS/BabylonNative/pull/30/files#diff-fec826feae04e51c0d94076385408bdcR207 )
For a local build, you have to do it manually. The goal of this task is to do it automaticaly like V8 nuget ( https://github.com/BabylonJS/BabylonNative/blob/master/Library/Dependencies/napi/CMakeLists.txt#L28 )
So just by opening the project with Android Studio, everything is setup and builds fine.

[CedricGuillemet]

Screen ratio is not updated with screen rotation

• Build Android project for your device
• Run on your device
• you should see the cube
• turn your device (portrait->landscape or lanscape->portrait)
• The image rotate but the display ratio is not correct. The cube is flatten.

Is this entry ( https://github.com/CedricGuillemet/BabylonNative/blob/AndroidRender/TestApp/Source/Android/app/src/main/cpp/BabylonNativeJNI.cpp#L111 ) is called when screen rotates?
If it does, maybe doing that call ( https://github.com/CedricGuillemet/BabylonNative/blob/AndroidRender/TestApp/Source/Android/app/src/main/cpp/BabylonNativeJNI.cpp#L106 ) will be enough
If not, there is at least a call to bgfx::init that does the initialization. Find the source of the dimension set for init and check/update the values.

Fixed! UpdateSize call when surfaceChanged was not called.

[CedricGuillemet]

Shader compliance and parsing between BJS and bgfx is not great for OpenGL. Expect issues when dealing with more complex shaders

Step 1:

• Android project builds and runs on your device
• uncomment 1 line in https://github.com/CedricGuillemet/BabylonNative/blob/AndroidRender/TestApp/Scripts/experience.js#L53 to test if the scene loads fine
• Fix what you can. Detail crash/error message here otherwise or can you also ping me.

Step 2:

• update gradle build here : https://github.com/CedricGuillemet/BabylonNative/blob/AndroidRender/TestApp/Source/Android/app/build.gradle#L51
  to copy .js files from https://github.com/CedricGuillemet/BabylonNative/tree/AndroidRender/TestApp/ValidationTests into the .apk
• Replace experience.js here https://github.com/CedricGuillemet/BabylonNative/blob/AndroidRender/TestApp/Source/Android/app/src/main/cpp/BabylonNativeJNI.cpp#L102 with viewport_test.js then framebuffer_setview_test.js, decode_encode_image_test.js, address_filtering_test.js, gltf2_asset_generator_test.js
• Compare with Win32 version ( https://github.com/CedricGuillemet/BabylonNative/blob/AndroidRender/TestApp/Source/Win32/App.cpp#L91 )
• Fix what you can. Detail crash/error message here otherwise.

Done for now. Shaders compile. Some issues with specular (float precision?)

[CedricGuillemet]

Add support for SSL

System libraries are used on Windows and Mac/Ios for SSL. Nothing is available on Android for native development. So, we need to add OpenSLL to the app.

Enable SSL here:
https://github.com/CedricGuillemet/BabylonNative/blob/AndroidRender/TestApp/Source/Android/app/build.gradle#L15
Add OpenSSL subrepo and compile the lib with cmake

PR is here : #113

one remaining problem with cmake alias