-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[CR130] Crash at Content Filtering #41402
Open
1 of 6 tasks
Labels
crash
OS/Android
Fixes related to Android browser functionality
QA/Test-Plan-Specified
QA/Yes
regression
release-notes/exclude
Comments
AlexeyBarabash
added
the
OS/Android
Fixes related to Android browser functionality
label
Oct 4, 2024
AlexeyBarabash
added a commit
to brave/brave-core
that referenced
this issue
Oct 4, 2024
fixes brave/brave-browser#41402 Crash happened at app menu => Settings => Brave Shields & privacy => Content Filtering. Tombstone ``` java.lang.ClassCastException: org.chromium.chrome.browser.settings.SettingsActivity cannot be cast to org.chromium.chrome.browser.settings.BraveSettingsActivity at org.chromium.chrome.browser.shields.ContentFilteringFragment.onAttach(ContentFilteringFragment.java:86) ``` Upstream had added a new place to create Intent for Settings activity. Related Chromium change: https://source.chromium.org/chromium/chromium/src/+/d4af6a2c4ccc6427aa7df0f2beacf61793459e66 [Settings] Introduce a central place to construct intents SettingsIntentUtil becomes the central place to construct intents launching the settings activity. Bug: b/356743945 Change-Id: Id6988af4a182fe6603bb02c9dae8b2ac48f80fed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5832868
AlexeyBarabash
added a commit
to brave/brave-core
that referenced
this issue
Oct 4, 2024
fixes brave/brave-browser#41402 Crash happened at app menu => Settings => Brave Shields & privacy => Content Filtering. Tombstone ``` java.lang.ClassCastException: org.chromium.chrome.browser.settings.SettingsActivity cannot be cast to org.chromium.chrome.browser.settings.BraveSettingsActivity at org.chromium.chrome.browser.shields.ContentFilteringFragment.onAttach(ContentFilteringFragment.java:86) ``` Upstream had added a new place to create Intent for Settings activity. Related Chromium change: https://source.chromium.org/chromium/chromium/src/+/d4af6a2c4ccc6427aa7df0f2beacf61793459e66 [Settings] Introduce a central place to construct intents SettingsIntentUtil becomes the central place to construct intents launching the settings activity. Bug: b/356743945 Change-Id: Id6988af4a182fe6603bb02c9dae8b2ac48f80fed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5832868
AlexeyBarabash
added a commit
to brave/brave-core
that referenced
this issue
Oct 4, 2024
fixes brave/brave-browser#41402 Crash happened at app menu => Settings => Brave Shields & privacy => Content Filtering. Tombstone ``` java.lang.ClassCastException: org.chromium.chrome.browser.settings.SettingsActivity cannot be cast to org.chromium.chrome.browser.settings.BraveSettingsActivity at org.chromium.chrome.browser.shields.ContentFilteringFragment.onAttach(ContentFilteringFragment.java:86) ``` Upstream had added a new place to create Intent for Settings activity. Related Chromium change: https://source.chromium.org/chromium/chromium/src/+/d4af6a2c4ccc6427aa7df0f2beacf61793459e66 [Settings] Introduce a central place to construct intents SettingsIntentUtil becomes the central place to construct intents launching the settings activity. Bug: b/356743945 Change-Id: Id6988af4a182fe6603bb02c9dae8b2ac48f80fed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5832868
24 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
crash
OS/Android
Fixes related to Android browser functionality
QA/Test-Plan-Specified
QA/Yes
regression
release-notes/exclude
Description
Regression after CR130.
Browser crashes when open Content Filtering.
Steps to reproduce
App menu => Settings => Brave Shields & privacy => Content Filtering
Actual result
crash
Expected result
no crash
Reproduces how often
Easily reproduced
Brave version
Own-built Nightly with CR130.
1.72.61
Device
any
Channel information
Reproducibility
Miscellaneous information
No response
The text was updated successfully, but these errors were encountered: