Commit 925179d
authored
Use Babylon Native's XMLHttpRequest (#94)
React Native's XMLHttpRequest uses too much memory and is too slow for use with Babylon.js. This change brings in Babylon Native's XMLHttpRequest, but does not replace the globally scoped one provided by React Native. Instead, it exposes it from the _native object, which WebRequest will specifically look for in @babylonjs/[email protected] or newer.
- Update Babylon.js to 4.2.0-beta.12.
- Update Babylon Native to latest (for corresponding XMLHttpRequest related changes).
- Add XMLHttpRequest to the Android and iOS build.
- Initialize XMLHttpRequest in the Android and iOS interop layers.
- Add XMLHttpRequest related libs to the pod file and package build validation.
Other unrelated changes included in this PR:
- Updating some GitHub build tasks to the latest to see if it resolves some confusing build warnings.
- Update iOS test app project files to use iOS 12 to match what the Babylon React Native podspec requires (otherwise I get errors when building Release in XCode 12).
- Revert to the old workaround for the promise continuation issue (it turns out the new approach doesn't work on iOS in Release builds).
- Add @react-native-community/cli as a dev dependency to the test apps as it seems we can use it to convert a Hermes profile to CDT profiler format and view it in Chrome (still trying to get this to work).1 parent 61a3a8f commit 925179d
File tree
19 files changed
+725
-39
lines changed- .github/workflows
- Apps
- PackageTest
- ios/PackageTest.xcodeproj
- Playground
- ios
- Playground.xcodeproj
- Modules/@babylonjs/react-native
- android
- src/main/cpp
- ios
- submodules
- Package
- iOS
19 files changed
+725
-39
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
| 12 | + | |
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
| |||
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
31 | | - | |
| 31 | + | |
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
| 11 | + | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
26 | | - | |
| 26 | + | |
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| |||
Lines changed: 4 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
178 | 178 | | |
179 | 179 | | |
180 | 180 | | |
181 | | - | |
182 | 181 | | |
183 | 182 | | |
184 | 183 | | |
| |||
554 | 553 | | |
555 | 554 | | |
556 | 555 | | |
557 | | - | |
| 556 | + | |
558 | 557 | | |
559 | 558 | | |
560 | 559 | | |
| |||
574 | 573 | | |
575 | 574 | | |
576 | 575 | | |
577 | | - | |
| 576 | + | |
578 | 577 | | |
579 | 578 | | |
580 | 579 | | |
| |||
784 | 783 | | |
785 | 784 | | |
786 | 785 | | |
787 | | - | |
| 786 | + | |
788 | 787 | | |
789 | 788 | | |
790 | 789 | | |
| |||
837 | 836 | | |
838 | 837 | | |
839 | 838 | | |
840 | | - | |
| 839 | + | |
841 | 840 | | |
842 | 841 | | |
843 | 842 | | |
| |||
0 commit comments