Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash in set Brave as a default browser #29428

Closed
SergeyZhukovsky opened this issue Mar 30, 2023 · 2 comments · Fixed by brave/brave-core#17916
Closed

Crash in set Brave as a default browser #29428

SergeyZhukovsky opened this issue Mar 30, 2023 · 2 comments · Fixed by brave/brave-core#17916
Assignees
Labels
crash OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/include

Comments

@SergeyZhukovsky
Copy link
Member

SergeyZhukovsky commented Mar 30, 2023

I found that crash in 1.49.x. We need fix it and uplift to a stable channel. For verifying just make sure that you can set Brave as a default browser via a dialog that appears on a 5th open of the browser.

java.lang.IllegalStateException: Can not perform this action after onSaveInstanceState
        at androidx.fragment.app.FragmentManager.checkStateLoss(FragmentManager.java:1610)
        at androidx.fragment.app.FragmentManager.enqueueAction(FragmentManager.java:1650)
        at androidx.fragment.app.BackStackRecord.commitInternal(BackStackRecord.java:341)
        at androidx.fragment.app.BackStackRecord.commit(BackStackRecord.java:306)
        at androidx.fragment.app.DialogFragment.show(DialogFragment.java:508)
        at org.chromium.chrome.browser.set_default_browser.BraveSetDefaultBrowserUtils.showBraveSetDefaultBrowserDialog(BraveSetDefaultBrowserUtils.java:112)
        at org.chromium.chrome.browser.set_default_browser.BraveSetDefaultBrowserUtils.checkSetDefaultBrowserModal(BraveSetDefaultBrowserUtils.java:71)
        at org.chromium.chrome.browser.app.BraveActivity.finishNativeInitialization(BraveActivity.java:997)
        at org.chromium.chrome.browser.ChromeTabbedActivity.finishNativeInitialization(ChromeTabbedActivity.java:1029)
        at org.chromium.base.CallbackController$CancelableRunnable.run(CallbackController.java:130)
        at android.os.MessageQueue.nativePollOnce(Native Method)
        at android.os.MessageQueue.next(MessageQueue.java:339)
        at android.os.Looper.loop(Looper.java:200)
        at android.app.ActivityThread.main(ActivityThread.java:8312)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:632)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1049)
@kjozwiak
Copy link
Member

The above requires 1.50.117 or higher for 1.50.x verification 👍

@Uni-verse
Copy link
Contributor

Uni-verse commented Apr 18, 2023

Verified on Samsung Galaxy S21 running Android 13 using the following version(s):

Brave	1.50.120 Chromium: 112.0.5615.121 (Official Build) (64-bit) 
Revision	39cc4e45904ae9f1741c4fbaa866e629c96f2268-refs/branch-heads/5615_51@{#8}
OS	Android 13; Build/TP1A.220624.014
  • Ensured that crash does not occur when setting brave as default browser on the 5th app launch.
  • Ensured that user can set default browser during the onboarding flow without issues.
  • Ensured that opening links work when Brave is set to default browser
default_crash1.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crash OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/include
Projects
None yet
5 participants