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 Report] Brave 1.68.134 crashes for sidepanel extensions on windows #40262

Closed
A-Sam opened this issue Aug 4, 2024 · 11 comments
Closed

Comments

@A-Sam
Copy link

A-Sam commented Aug 4, 2024

Test plan

  1. Fresh profile
  2. Install https://chromewebstore.google.com/detail/tab-tag-tree/lnlodalopnihdjkilbkpijndkddfoemc
  3. Click the extension button and pin the extension to the top
  4. Show Brave's sidebar if it's not already visible
  5. Right click the Brave Talk and Brave Wallet icons and choose Remove on each
  6. Close Brave
  7. Re-open Brave
  8. Click the extension action button

Description

IMPORTANT: Your crash has already been automatically reported to our crash system. Please file this bug only if you can provide more information about it.

Brave Version: 1.68.134 Chromium: 127.0.6533.88
Operating System: Windows NT 10.0.22631

URL (if applicable) where crash occurred:
Any extension that uses sidepanel:
https://chromewebstore.google.com/detail/tab-tag-tree/lnlodalopnihdjkilbkpijndkddfoemc
https://chromewebstore.google.com/detail/copilot-sidebar-for-chrom/ncjedehfkpnliaafimjhdjjeggmfmlgf

Can you reproduce this crash?

What steps will reproduce this crash? (If it's not reproducible, what were you doing just before the crash?)

  1. Install from above any extension that uses sidepanel
  2. Open the extension (make sure sidepanel is allowed)
  3. Brave crashes instantly

Reports:
dccd0d00-ea85-600c-0000-000000000000
d5cd0d00-ea85-600c-0000-000000000000

DO NOT CHANGE BELOW THIS LINE
Crash ID: crash/dccd0d00-ea85-600c-0000-000000000000

@github-actions github-actions bot added the crash label Aug 4, 2024
@A-Sam
Copy link
Author

A-Sam commented Aug 4, 2024

This might be related: #31328

@fabivs
Copy link

fabivs commented Aug 5, 2024

I am experiencing the same behavior on Windows while trying to open this extension's sidebar: https://github.com/killergerbah/asbplayer
Instant crash in any kind of circumstance, as soon as I try to open the extension sidebar.
Does not happen on Mac, with the same version of Brave and the same set of extensions installed.

@bsclifton bsclifton added feature/extensions priority/P2 A bad problem. We might uplift this to the next planned release. labels Aug 5, 2024
@fabivs
Copy link

fabivs commented Aug 6, 2024

I was able to find a workaround as described here:
#39971

@bsclifton
Copy link
Member

bsclifton commented Aug 6, 2024

Thanks @fabivs!

@A-Sam can you try this too? (as a temporary work-around)

For me crash is caused because I removed brave talk and brave wallet from sidebar, adding them back fixed the issue

@A-Sam
Copy link
Author

A-Sam commented Aug 6, 2024

Hi @bsclifton, thanks for the suggestion. Yes it works now with Brave talk and wallet.

@bsclifton bsclifton self-assigned this Aug 6, 2024
@bsclifton
Copy link
Member

Confirmed this works as expected on Beta (1.69) and Nightly (1.70) and also confirmed it crashes on Release (1.68).

Will dig in some more and try to find the root cause! Thanks for reporting @A-Sam 😄

@bsclifton
Copy link
Member

OK found that we fixed this already in Beta and Nightly with brave/brave-core#24362

Will create an uplift for Release channel

@LaurenWags
Copy link
Member

Closing as this issue is resolved by brave/brave-core#24362.

Requires 1.68.140 or higher to test 👍🏻

@GeetaSarvadnya
Copy link

Verification PASSED on

Brave | 1.68.140 Chromium: 127.0.6533.100 (Official Build) (64-bit)
-- | --
Revision | 612f5cee6491be373e8133bcda3f56522fbe621e
OS | Windows 10 Version 22H2 (Build 19045.4651)

Reproduced the issue on 1.68.137 Chromium: 127.0.6533.10 upgraded the profile to 1.68.140 Chromium: 127.0.6533.100 and ensured that click on the extension button doesn't crash the brave
image

@A-Sam
Copy link
Author

A-Sam commented Aug 15, 2024

Tested after the update and confirming the fix. Thanks @bsclifton @LaurenWags

@LaurenWags
Copy link
Member

Appreciate the confirmation @A-Sam 👍🏻

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

No branches or pull requests

6 participants