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

[Android] Crash on certain devices on custom tabs #9857

Closed
SergeyZhukovsky opened this issue May 20, 2020 · 1 comment · Fixed by brave/brave-core#5604
Closed

[Android] Crash on certain devices on custom tabs #9857

SergeyZhukovsky opened this issue May 20, 2020 · 1 comment · Fixed by brave/brave-core#5604

Comments

@SergeyZhukovsky
Copy link
Member

Description

There is a crash on beta channel on 1.9.x related to Custom Tabs. Not sure how to describe that but certain devices crash while others don't. I had a device Samsung Galaxy S10+ that experienced that crash but only on release version. I verified everything on it while fixing in release mode.
2020-05-19 20:33:42.558 29508-29508/? W/System.err: java.lang.ClassCastException: android.view.ContextThemeWrapper cannot be cast to android.app.Activity 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at x20.a(PG:62) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at org.chromium.chrome.browser.preferences.website.BraveShieldsContentSettings.blockedEvent(PG:26) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at android.os.MessageQueue.nativePollOnce(Native Method) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at android.os.MessageQueue.next(MessageQueue.java:336) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at android.os.Looper.loop(Looper.java:197) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at android.app.ActivityThread.main(ActivityThread.java:8016) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at java.lang.reflect.Method.invoke(Native Method) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493) 2020-05-19 20:33:42.559 29508-29508/? W/System.err: at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1076) 2020-05-19 20:33:42.560 29508-29508/? A/chromium: [FATAL:jni_android.cc(249)] Please include Java exception stack in crash report

Steps to reproduce

  1. Install beta 1.9.69
  2. Open custom tab via opening a link on gmail or from slack

Actual result

the browser crashes

Expected result

Issue reproduces how often

On my problem device I can replicate it every time, other devices I couldn't

Issue happens on

  • Current Play Store version?
  • Beta build? yes

Device details

  • Install type (ARM, x86):
  • Device (Phone, Tablet, Phablet):
  • Android version:

Brave version

Website problems only

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

Additional information

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented May 22, 2020

Verification passed on Samsung Galaxy J3 version 5.1.1 running BraveModernarm.apk 1.9.74

  • Could not reproduce the issue in 1.9.69 Android beta version, no crash when custom tabs are opened via the slack link or Gmail link
  • No crash when custom tabs are opened via the slack link or Gmail link in 1.9.74

Verification passed on Asus Zenfone (x86) with Android 6 running BraveModernx86.apk 1.9.74

  • Could not reproduce the issue in 1.9.69 Android beta version, no crash when custom tabs are opened via the slack link or Gmail link
  • No crash when custom tabs are opened via the slack link or Gmail link in 1.9.74

Verification passed on BraveModernarm v1.9.74 on Samsung Galaxy Tab S5e (Android 9.0)

  • Could not reproduce the issue in 1.9.69 Android beta version, no crash when custom tabs are opened via the slack link or Gmail link
  • No crash when custom tabs are opened via the slack link or Gmail link in 1.9.74

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

Successfully merging a pull request may close this issue.

5 participants