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

pressing on "Private" crashes brave while using private tabs when "#enable-new-profile-picker" has been enabled #13822

Closed
LUC18fknU7P opened this issue Jan 28, 2021 · 6 comments

Comments

@LUC18fknU7P
Copy link

Description

pressing the button described in the image below crashes brave and causes all tabs to close instantly
image

Steps to Reproduce

  1. Open private tab
  2. Press the "Private" button
  3. The browser should crash.

Actual result:

N/A

Expected result:

Browser resumes operation as expected

Reproduces how often:

Every time the button is pressed

Brave version (brave://version info)

Name Value
Brave 1.19.86 Chromium: 88.0.4324.96 (Official Build) (64-bit)
Revision 68dba2d8a0b149a1d3afac56fa74648032bcf46b-refs/branch-heads/4324@{#1784}
OS Windows 10 OS Version 2004 (Build 19041.630)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Haven't tried
  • Can you reproduce this issue with the nightly channel? Haven't tried

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? No
  • Does the issue resolve itself when disabling Brave Rewards? No
  • Is the issue reproducible on the latest version of Chrome? Presumably not.

Miscellaneous Information:

#enable-profiles-ui-revamp is enabled on brave://flags

@bridiver
Copy link
Contributor

I can't repro on current release or nightly, can you try updating?

@LUC18fknU7P
Copy link
Author

I can't repro on current release or nightly, can you try updating?

Did you enable #enable-profiles-ui-revamp? others are also crashing with it enabled

@wsor4035
Copy link

also crashes when #enable-new-profile-picker is enabled, #13102 (comment)

@rebron rebron added crash needs-investigation A bug not 100% confirmed/fixed labels Jan 29, 2021
@kjozwiak kjozwiak changed the title Bug: Pressing on "Private" crashes brave while using private tabs pressing on "Private" crashes brave while using private tabs when "#enable-new-profile-picker" has been enabled Jan 30, 2021
@kjozwiak kjozwiak removed the needs-investigation A bug not 100% confirmed/fixed label Jan 30, 2021
@kjozwiak
Copy link
Member

kjozwiak commented Jan 30, 2021

As per the above, you'll need to enable #enable-profiles-ui-revamp via brave://flags. STR:

  • launch brave and enable #enable-profiles-ui-revamp via brave://flags (restart)
  • open a new private tab and click on the Private tab profile picker
  • you'll get the following crash: 8b1d0000-9cc8-c705-0000-000000000000

You'll get the following crash:

[ 00 ] ui::ThemedVectorIcon::GetImageSkia(ui::NativeTheme const *)
[ 01 ] views::View::GetNativeTheme()
[ 02 ] views::ThemedVectorIconBackground::Paint(gfx::Canvas *,views::View *)
[ 03 ] views::View::OnPaintBackground(gfx::Canvas *)
[ 04 ] SkCanvas::clipRect(SkRect const &,SkClipOp,bool)
[ 05 ] SkNoDrawCanvas::SkNoDrawCanvas(int,int)
[ 06 ] views::View::OnPaint(gfx::Canvas *)
[ 07 ] ui::PaintRecorder::PaintRecorder(ui::PaintContext const &,gfx::Size const &,float,float,ui::PaintCache *)
[ 08 ] views::View::Paint(views::PaintInfo const &)
[ 09 ] RtlFreeHeap
[ 10 ] 0x5400000120
[ 11 ] RtlUnwindEx
[ 12 ] RtlUnwindEx

@LUC18fknU7P
Copy link
Author

I've recently also crashed by pressing "Allow" on consent popups e.g. push notifications

@mariospr
Copy link
Contributor

I just managed to reproduce this by enabling #enable-new-profile-picker on a built-from-sources Brave off the master branch, but can't get the crash anymore on the cr92 branch, could this have been fixed upstream?

@iefremov iefremov added closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. closed/works-for-me QA/No release-notes/exclude and removed closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. labels Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants