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

[Desktop] Browser Crash when loading brave://settings/content/all #10160

Closed
btlechowski opened this issue Jun 9, 2020 · 4 comments · Fixed by brave/brave-core#5777
Closed

Comments

@btlechowski
Copy link

btlechowski commented Jun 9, 2020

Follow up to #10049

Important: This is a regression introduced in 1.10.x
Important: The browser can get into crash loop if the page is automatically opened after restart

Steps to Reproduce

  1. Clean profile
  2. Open brave://settings/content/all

Actual result:

Crash

Expected result:

No Crash

Reproduces how often:

100% repro rate

Brave version (brave://version info)

Reproducible on

Brave 1.10.86 Chromium: 83.0.4103.61 (Official Build) (64-bit)
Revision 94f915a8d7c408b09cc7352161ad592299f384d2-refs/branch-heads/4103@{#561}
OS Ubuntu 18.04 LTS
Brave 1.11.56 Chromium: 83.0.4103.61 (Official Build) dev (64-bit)
Revision 94f915a8d7c408b09cc7352161ad592299f384d2-refs/branch-heads/4103@{#561}
OS Windows 7 Service Pack 1 (Build 7601.24544)
Brave 1.12.8 Chromium: 83.0.4103.61 (Official Build) nightly (64-bit)
Revision 94f915a8d7c408b09cc7352161ad592299f384d2-refs/branch-heads/4103@{#561}
OS Windows 7 Service Pack 1 (Build 7601.24544)

cc @brave/legacy_qa @mkarolin @rebron @bsclifton

@kjozwiak
Copy link
Member

kjozwiak commented Jun 9, 2020

Reproduced the issue on macOS 10.15.5 x64 Catalina using the following build:

Brave | 1.10.86 Chromium: 83.0.4103.61 (Official Build) (64-bit)
-- | --
Revision | 94f915a8d7c408b09cc7352161ad592299f384d2-refs/branch-heads/4103@{#561}
OS | macOS Version 10.15.5 (Build 19F101)

I'm going to move this into 1.10.x as a release/blocker as we never want to introduce new crashes. Looks like @mkarolin is already working on a fix as per brave/brave-core#5777.

CCing @rebron @bsclifton @bbondy.

@kjozwiak kjozwiak added this to the 1.10.x - Release milestone Jun 9, 2020
mkarolin added a commit to brave/brave-core that referenced this issue Jun 9, 2020
Fixes brave/brave-browser#10160

Updated code to use a patch instead of the override for a function
because the function is now called from the same file where it is
defined.
@LaurenWags
Copy link
Member

labeling as QA/Blocked until we have a build which contains this fix

@kjozwiak
Copy link
Member

BTW, nice find @btlechowski 👍

@kjozwiak
Copy link
Member

kjozwiak commented Jun 11, 2020

Verification PASSED on macOS 10.15.5 x64 using the following build:

Brave | 1.10.88 Chromium: 83.0.4103.97 (Official Build) (64-bit)
-- | --
Revision | 326d148b9655369b86498d9ecca39f63dd2bdd2d-refs/branch-heads/4103@{#657}
OS | macOS Version 10.15.5 (Build 19F101)
  • ensured that loading brave://settings/content/all doesn't crash the browser

Verification passed on

Brave | 1.10.88 Chromium: 83.0.4103.97 (Official Build) (64-bit)
-- | --
Revision | 326d148b9655369b86498d9ecca39f63dd2bdd2d-refs/branch-heads/4103@{#657}
OS | Windows 10 OS Version 1803 (Build 17134.1006)

  • ensured that loading brave://settings/content/all doesn't crash the browser

Verification passed on

Brave 1.10.88 Chromium: 83.0.4103.97 (Official Build) (64-bit)
Revision 326d148b9655369b86498d9ecca39f63dd2bdd2d-refs/branch-heads/4103@{#657}
OS Ubuntu 18.04 LTS
  • verified that loading brave://settings/content/all doesn't crash the browser

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