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 after clicking on Sync, About Brave and Settings menus in 1.17.70 #12759

Closed
GeetaSarvadnya opened this issue Nov 18, 2020 · 10 comments

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 18, 2020

Description

Brave crashes after clicking on Sync, About Brave and Settings menus in 1.17.70

Steps to Reproduce

  1. Clean or upgraded profile 1.17.70
  2. Open hamburger menu, click on Sync, About Brave and Settings menus
  3. Brave crashes

Actual result:

Crash

Expected result:

No crash

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.17.70 Chromium: 87.0.4280.60 (Official Build) (64-bit)
Revision 12697cfeb273d7de95cf9b18350d2c457f58224c-refs/branch-heads/4280@{#1352}
OS Windows 10 OS Version 2004 (Build 19041.572)

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the nightly channel? No

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @brave/legacy_qa @rebron @bsclifton

@AlexeyBarabash
Copy link
Contributor

Maybe it can be related with CHECK mentioned in brave/brave-core#7121 (comment)

https://source.chromium.org/chromium/chromium/src/+/master:content/public/browser/web_ui_message_handler.h;l=125;drc=f50df03b5c021e63d80ecf09ea452fa9976071ea;bpv=1;bpt=1?q=content%2Fpublic%2Fbrowser%2Fweb_ui_message_handler.h&ss=chromium%2Fchromium%2Fsrc

  template <typename... Values>
  void CallJavascriptFunction(const std::string& function_name,
                              const Values&... values) {
    CHECK(IsJavascriptAllowed()) << "Cannot CallJavascriptFunction before "
                                    "explicitly allowing JavaScript.";

But for me it wasn't always reproducible.

I noticed it after updating my repo so it has Upgrade from Chromium 87.0.4280.49 to Chromium 87.0.4280.60

@LaurenWags
Copy link
Member

LaurenWags commented Nov 18, 2020

Reproduces on macOS with the below build

Brave 1.17.70 Chromium: 87.0.4280.60 (Official Build) (x86_64)
Revision 12697cfeb273d7de95cf9b18350d2c457f58224c-refs/branch-heads/4280@{#1352}
OS macOS Version 10.14.6 (Build 18G6042)

Crash Report IDs:
11a70000-0b46-6205-0000-000000000000
13a70000-0b46-6205-0000-000000000000
18a70000-0b46-6205-0000-000000000000

Could not reproduce with previous 1.17.x build

Brave 1.17.69 Chromium: 87.0.4280.60 (Official Build) (x86_64)
Revision 12697cfeb273d7de95cf9b18350d2c457f58224c-refs/branch-heads/4280@{#1352}
OS macOS Version 10.14.6 (Build 18G6042)

Could not reproduce with Nightly

Brave 1.19.15 Chromium: 87.0.4280.63 (Official Build) nightly (x86_64)
Revision 6813efad2f088f979317946a8425f16a7028fa43-refs/branch-heads/4280@{#1410}
OS macOS Version 10.14.6 (Build 18G6042)

@LaurenWags LaurenWags added this to the 1.17.x - Release milestone Nov 18, 2020
@GeetaSarvadnya
Copy link
Author

Crash Report Id's collected from Windows 10 x64 - 1.17.70

8ba70000-0b46-6205-0000-000000000000
fca70000-0b46-6205-0000-000000000000
06a80000-0b46-6205-0000-000000000000

@marcmv1
Copy link

marcmv1 commented Nov 18, 2020

Same behavior for me on 1.17.70. Windows 10 20H2, 64 bit Brave.

@marcmv1
Copy link

marcmv1 commented Nov 18, 2020

Another data point... my browser string is still reading .60, the commit log says it was upgraded to .66.

Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.60 Safari/537.36

@srirambv
Copy link
Contributor

Need to retest #12699 once this is fixed for NTP settings

@kjozwiak
Copy link
Member

Closing as brave/brave-core#7183 & brave/brave-core#7184 were merged.

@jorgicio
Copy link

I'm having the same issue in Linux.

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Nov 19, 2020

Verification passed on


Brave | 1.17.71 Chromium: 87.0.4280.66 (Official Build) (64-bit)
-- | --
Revision | fd98a29dd59b36f71e4741332c9ad5bda42094bf-refs/branch-heads/4280@{#1432}
OS | Windows 10 OS Version 2004 (Build 19041.572)

  • Verified STR from the description and ensured there is no crash after clicking on Sync, Settings and About Brave menus
  • Verified all the menu options from the 3 dot menu ( Customize and control brave) options and ensured there is no crash
  • Verified PT/TOR/Guest window and ensured all the tabs are working as expected
  • Verified few settings randomly in brave://settings and ensured settings are working as expected

Verification passed on

Brave 1.17.71 Chromium: 87.0.4280.66 (Official Build) (64-bit)
Revision fd98a29dd59b36f71e4741332c9ad5bda42094bf-refs/branch-heads/4280@{#1432}
OS Ubuntu 18.04 LTS
  • Verified STR from the description and ensured there is no crash after clicking on Sync, Settings and About Brave menus
  • Verified all the menu options from the 3 dot menu ( Customize and control brave) options and ensured there is no crash
  • Verified PT/TOR/Guest window and ensured all the tabs are working as expected
  • Verified few settings randomly in brave://settings and ensured settings are working as expected
  • Verified upgrade scenarios from 1.16.x. Made sure no crash when settings pages from previous session are loaded

Verified passed with

Brave | 1.17.71 Chromium: 87.0.4280.66 (Official Build) (x86_64)
-- | --
Revision | fd98a29dd59b36f71e4741332c9ad5bda42094bf-refs/branch-heads/4280@{#1432}
OS | macOS Version 10.14.6 (Build 18G6042)
  • Verified STR from the description and ensured there is no crash after clicking on Sync, Settings and About Brave menus

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