-
Notifications
You must be signed in to change notification settings - Fork 220
Conversation
@Softvision-GeluHaiduc You can start with the language testing after this lands. |
Got this crash on first run:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I get this crash on launch:
E/GeckoCrashHandler: >>> REPORTING UNCAUGHT EXCEPTION FROM THREAD 1 ("main")
java.lang.NullPointerException: Attempt to invoke interface method 'boolean java.util.List.isEmpty()' on a null object reference
at org.mozilla.vrbrowser.utils.LocaleUtils.getPreferredLanguages(LocaleUtils.java:161)
at org.mozilla.vrbrowser.utils.LocaleUtils.getPreferredLanguageTags(LocaleUtils.java:151)
at org.mozilla.vrbrowser.VRBrowserActivity.onCreate(VRBrowserActivity.java:245)
at android.app.Activity.performCreate(Activity.java:6724)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1119)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2632)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2740)
at android.app.ActivityThread.-wrap12(ActivityThread.java)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1477)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:154)
at android.app.ActivityThread.main(ActivityThread.java:6144)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)
Did a clean build and got same result.
I get the same crash after a clean install also. |
I was testing on the Quest. |
Fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The telemetry tests failed for this PR. So we should probably figure that out or see if it was a one off failure.
@daoshengmu Said he would investigate the test failures. |
It works with t-c now. :) |
I have verified these new behaviors using the latest RC, Firefox Reality version 9 RC1 (33f6c4d), installed on the Oculus Go, Quest, HTC Vive Focus, Focus Plus and Pico Neo2 headsets. |
@Softvision-GeluHaiduc This two PRs should fix those issues: |
This adds support for a "Follow system language" placeholder that uses whatever the system locale is as long as it's supported by FxR, otherwise it falls back to en-US.
Also fixes some issues with the correct language hot reload. Now we should be able to change the language either from the Settings or from the system settings and everything so be instantly localized in the new language.
Things to test: