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

Crash related to Brave News on NTP #27753

Closed
SergeyZhukovsky opened this issue Jan 11, 2023 · 2 comments · Fixed by brave/brave-core#16617
Closed

Crash related to Brave News on NTP #27753

SergeyZhukovsky opened this issue Jan 11, 2023 · 2 comments · Fixed by brave/brave-core#16617
Assignees
Labels
crash OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/exclude

Comments

@SergeyZhukovsky
Copy link
Member

SergeyZhukovsky commented Jan 11, 2023

I found that crash in GPS on a stable channel 1.46.x. https://github.com/brave/brave-core/blob/1.46.x/android/java/org/chromium/chrome/browser/ntp/BraveNewTabPageLayout.java#L293. Perhaps a check on activity has to be done inside the handler and not outside?

Crash stack:

Exception java.lang.NullPointerException: Attempt to invoke virtual method 'org.chromium.chrome.browser.tab.Tab org.chromium.chrome.browser.app.ChromeActivity.E3()' on a null object reference
  at org.chromium.chrome.browser.ntp.BraveNewTabPageLayout.lambda$onAttachedToWindow$0 (BraveNewTabPageLayout.java:293)
  at org.chromium.chrome.browser.ntp.BraveNewTabPageLayout$$InternalSyntheticLambda$1$48595ce0a6477504149fb7e91db1bae31bc3c267dc5e1c3a14d4f7f21f0313a6$0.run$bridge (BraveNewTabPageLayout.java:173)
  at android.os.Handler.handleCallback (Handler.java:883)
  at android.os.Handler.dispatchMessage (Handler.java:100)
  at android.os.Looper.loop (Looper.java:237)
  at android.app.ActivityThread.main (ActivityThread.java:7811)
  at java.lang.reflect.Method.invoke (Method.java)
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:493)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1068)

STR/Cases:

As discussed during the weekly Android planning/triage meeting, a quick check should be sufficient enough:

  • ensure that the modal/card is appearing via NTP
  • ensure that you can opt-in into Brave News
  • ensure that you can tap/open articles within the news feed without any issues
  • add/remove a few sources and ensure nothing obvious has regressed
@kjozwiak
Copy link
Member

The above requires 1.47.171 or higher for 1.47.x verification 👍

@Uni-verse
Copy link
Contributor

Uni-verse commented Jan 12, 2023

Verified on Samsung Galaxy S21 using version:

Brave	1.47.171 Chromium: 109.0.5414.87 (Official Build) (64-bit) 
Revision	2dc18eb511c56e012081b4abc9e38c81c885f7d4-refs/branch-heads/5414@{#1241}
OS	Android 13; Build/TP1A.220624.014
  • Verified user can opt into brave news and load content in other tabs.
  • Verified the following use cases for brave news.
  • ensure that the modal/card is appearing via NTP
  • ensure that you can opt-in into Brave News
  • ensure that you can opt-in into Brave News via settings menu
  • ensure that you can tap/open articles within the news feed without any issues
  • add/remove a few sources and ensure nothing obvious has regressed
  • Brave news feed on upgraded profile
Example Example Example Example
screenshot-1673537019690 screenshot-1673537040994 screenshot-1673537524216 screenshot-1673537677029

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 priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants