ungoogled-software / ungoogled-chromium-android

Android build for ungoogled-chromium
GNU General Public License v3.0
495 stars 42 forks source link

Browser crashes on closing tabs from previous session(if the tabs are closed without loading them again) #59

Closed hbarsaiyan closed 3 years ago

hbarsaiyan commented 3 years ago

Please at least answer all questions marked with * below. Issues without answering these questions will be closed.

*Device info (please complete the following information):

*Is the bug related to extensions? Maybe 😅 I tried both arm and arm64. It happens on both.

*If not related to extensions, can the bug be reproduced with corresponding Chromium version? Yes/No

*Describe the bug If I open browser and close tabs from previous session without loading them again, the browser crashes.

*Expected behavior The browser should not crash.

Additional context I have attatched the crash log. Log_2021-02-16_23-16-09.txt

wchen342 commented 3 years ago

Can you describe in detail what the exact situation was? How many tabs are opened? How did you exit the browser? Did the first tab appear after you reopen the browser get restored? I am not sure what you mean by "without loading them again" because the tabs should be restored on re-opening the browser, how did you stop that from happening?

wchen342 commented 3 years ago

Ok I think I got what you meant. Did you mean that you were closing those tabs that are behind the first tab upon restoration? I think I reproduced that crash.

hbarsaiyan commented 3 years ago

Ok I think I got what you meant. Did you mean that you were closing those tabs that are behind the first tab upon restoration? I think I reproduced that crash.

Yes that is what I meant. If I close any other tab from previous session (apart from the one loading in foreground ) the browser crashes.

wchen342 commented 3 years ago

Ok. It was a problem I thought had been fixed. I will release an update soon.

wchen342 commented 3 years ago

Should be fixed in 9c184f7. The release is building.

EDIT: release is out.