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

Brave crashes when visiting brave://settings when Sync disabled via brave://flags #25156

Closed
specter78 opened this issue Sep 2, 2022 · 6 comments · Fixed by brave/brave-core#15031
Assignees

Comments

@specter78
Copy link

specter78 commented Sep 2, 2022

Test plan

See brave/brave-core#15031

Description

Brave v1.43.88 crashes when Brave Sync V2 flag is disabled.

On windows - it crashes on launch

On macOS- it crashes on opening preferences

@rebron
Copy link
Collaborator

rebron commented Sep 2, 2022

@specter78 Can you go into brave://crashes send the report and post the crash id in the thread if you get a chance.

@rebron rebron added the crash label Sep 2, 2022
@MadhaviSeelam
Copy link

MadhaviSeelam commented Sep 2, 2022

Reproduced this issue in Win 11 x64

Brave | 1.43.85 Chromium: 105.0.5195.68 (Official Build) (64-bit)
-- | --
Revision | ad13e82529051bac6a0e65f455e6d7a1e5fd7938-refs/branch-heads/5195@{#903}
OS | Windows 11 Version 21H2 (Build 22000.856)

Crash Report ID: | a72d0300-74c0-c508-0000-000000000000

  • install 1.43.88
  • open brave://flags and disable sync v2 and relaunch the browser
  • open brave://settings/ in a new tab

@bsclifton
Copy link
Member

bsclifton commented Sep 2, 2022

Also crashes on NTP when turning off brave://flags/#brave-news (as of 1.43)

We should probably remove these two flags because they've outlived their use. We can remove the feature flag level checks too. There is a specific preference now for News enabled

@bsclifton bsclifton added the priority/P2 A bad problem. We might uplift this to the next planned release. label Sep 2, 2022
@bsclifton bsclifton self-assigned this Sep 6, 2022
@bsclifton bsclifton changed the title Brave crash when certain flags are set Brave crashes when visiting brave://settings when Sync disabled via brave://flags Sep 7, 2022
@bsclifton
Copy link
Member

Narrowed this down - this first appeared in 1.43.84 (1.43.83 was fine)

@mkarolin this may be related to the l10n updates - I'm not sure if it'll crash if a string is missing? I might need help narrowing it down.

Here's a quick link to the commit log for 1.43.84 to help see what commits were added:
https://github.com/brave/brave-core/commits/40863c335718591dcccc3537d04c273425fc3677

@kjozwiak
Copy link
Member

kjozwiak commented Sep 9, 2022

The above requires 1.43.92 or higher for 1.43.x verification 👍

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 12, 2022

Verification PASSED on

Brave | 1.43.92 Chromium: 105.0.5195.102 (Official Build) (64-bit)
-- | --
Revision | 4c16f5ffcc2da70ee2600d5db77bed423ac03a5a-refs/branch-heads/5195_55@{#4}
OS | Windows 10 Version 21H2 (Build 19044.1889)

Verified the test plan from brave/brave-core#15031
Reproduced the issue in 1.43.88
Confirmed that there is no crash when brave://settings opened once brave://flags/#brave-sync-v2 has been disabled in 1.43.92

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants