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

OSX accessibility zoom features crash brave #8149

Closed
cazssiew opened this issue Feb 9, 2020 · 11 comments
Closed

OSX accessibility zoom features crash brave #8149

cazssiew opened this issue Feb 9, 2020 · 11 comments

Comments

@cazssiew
Copy link

cazssiew commented Feb 9, 2020

Description

Using the OSX zoom features (ctrl+scroll or ctrl+hover text) crashes brave

Steps to Reproduce

  1. enable zoom and large hover text in system preferences > accessibility > zoom
  2. use in brave according to the selected hotkeys

Actual result:

brave crashes

Expected result:

zooming in/out / large hover text

Reproduces how often:

most often crashes instantly, occasionally takes a few seconds, but always ends up crashing.

Brave version (brave://version info)

Brave | 1.3.113 Chromium: 80.0.3987.87 (Official Build) (64-bit)
Revision | 449cb163497b70dbf98d389f54e38e85d4c59b43-refs/branch-heads/3987@{#801}
OS | macOS Version 10.15.3 (Build 19D76)

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? don't know
  • Can you reproduce this issue with the dev channel? don't know
  • Can you reproduce this issue with the nightly channel? don't know

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? no
@rebron
Copy link
Collaborator

rebron commented Feb 10, 2020

Easily reproduces on macOS Catalina. Not an issue on Mojave.
Crash ID: 1b43c4fb7d79705f

@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Feb 10, 2020
@kjozwiak
Copy link
Member

kjozwiak commented Feb 10, 2020

Thanks for the report @cazssiew 👍 Much appreciated. We'll make sure this gets addressed/fixed in the next release which is typically when we have a minor chromium bump.

Just @cazssiew, was this reproducible with 1.2.x or was this a recent regression?

@cazssiew
Copy link
Author

Awesome, thanks! Unfortunately I only started using the macos zoom very recently so I'm not sure. Sorry I can't be of more help.

@rebron
Copy link
Collaborator

rebron commented Feb 11, 2020

Reproducible on 1.2.43 using a fresh profile.

@mkarolin
Copy link
Contributor

@rebron could you, please, try with the Nightly Version 1.11.31 (Chromium 83). It didn't crash for me with cr83 branch build.

@Brave-Matt
Copy link

+1 from Community:
https://community.brave.com/t/brave-crashes-when-hover-text-is-locked-in-macos-catalina/202569/3

@rebron can we get an update on this? I'm sure there are other users seeing this as well.

@iefremov
Copy link
Contributor

@Brave-Matt Should be fixed in Nighlty with brave/brave-core#7832

@iefremov iefremov self-assigned this Feb 10, 2021
@iefremov
Copy link
Contributor

The fix brave/brave-core#7832 was uplifted to beta and release

@LaurenWags
Copy link
Member

LaurenWags commented Feb 18, 2021

@iefremov while testing I saw the app crash when attempting to reproduce using 1.19.x. Using 1.20.104 the app doesn't crash anymore but it seems crash reports are still being generated (quite generously) in brave://crashes. Examples:
94070000-ab7a-e305-0000-000000000000
92070000-ab7a-e305-0000-000000000000
95070000-ab7a-e305-0000-000000000000
91070000-ab7a-e305-0000-000000000000

Brave | 1.20.104 Chromium: 88.0.4324.152 (Official Build) (x86_64)
-- | --
Revision | 6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS | macOS Version 10.15.7 (Build 19H512)

@iefremov
Copy link
Contributor

this is expected @LaurenWags , all good.

@LaurenWags
Copy link
Member

LaurenWags commented Feb 18, 2021

Verified passed with macOS x64 10.15.7 (Catalina)

Brave | 1.20.104 Chromium: 88.0.4324.152 (Official Build) (x86_64)
-- | --
Revision | 6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS | macOS Version 10.15.7 (Build 19H512)

Verified using STR from #8149 (comment). No app crashes observed when using keyboard shortcuts for zoom in/out and large hover text. Note, crash reports observed as noted in #8149 (comment).


Verified passed with macOS x64 10.14.6 (Mojave)

Brave	1.20.104 Chromium: 88.0.4324.152 (Official Build) (x86_64)
Revision	6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS	macOS Version 10.14.6 (Build 18G103)

Verified using STR from #8149 (comment). No crashes observed in the app or on brave://crashes. Note, Mojave does not contain the large hover text setting. Testing was completed without that setting enabled, only the keyboard shortcuts for zoom were tested.


Verified (as much as possible) using STR from #8149 (comment). Wasn't able to trigger any crashes. Big Sur (11.2.1) does have large hover text setting, which I enabled, as well as used text zoom, in tandem, on both 1.19.92 as well as

Brave 1.20.104 Chromium: 88.0.4324.152 (Official Build) (x86_64)
Revision 6579930fc53b4dc589c042bec9d0a3778326974d-refs/branch-heads/4324@{#2106}
OS macOS Version 11.2.1 (Build 20D74)

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

9 participants