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

old favicons not being updated to v2 via brave://rewards after uprade - follow up to #37941 #38115

Open
kjozwiak opened this issue May 7, 2024 · 1 comment
Assignees
Labels
feature/rewards OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include

Comments

@kjozwiak
Copy link
Member

kjozwiak commented May 7, 2024

Description

Due to #37941, we had a bug where favicons weren't being displayed in the rewards panels and brave://rewards due to loading from chrome://favicon rather than chrome://favicon2 as per #37941 (comment). However, the icons are still appearing broken via brave://rewards unless a user visits the affected websites and pulls/downloads the new favicons.

@hffvld also experienced the above on Android and provided screenshots which have also been added into the issue.

Steps to Reproduce

  1. install 1.65.123 Chromium: 124.0.6367.91 and enable rewards
  2. visit several websites and notice that there's broken favicons appearing under brave://rewards
  3. update Brave to the latest version and revisit brave://rewards (you'll notice that the icons are still broken)

However, once you revisit the websites that are being listed under brave://rewards, the new favicons will be updated.

Actual result:

Desktop Example Android Example
image (1) 2024-05-06_16-01-23

Expected result:

Brave should attempt to check to see if there's any broken favicons and update them automatically rather than the user needing to revisit the website to pull the new ones from chrome://favicon2

Reproduces how often:

100% reproducible when using the STR/Cases outlined above.

Desktop Brave version:

Reproduced on Win 11 x64 using 1.65.123 Chromium: 124.0.6367.91 --> Upgrade --> 1.65.126 Chromium: 124.0.6367.118

Android Device details:

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

Version/Channel Information:

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

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? Only occurs for users that have rewards enabled and before Favicons are not displaying in the rewards panel #37941
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

@hffvld
Copy link
Contributor

hffvld commented May 7, 2024

I observed the same issue on Pixel 7 while testing Brave Nightly 1.67.70.

Device/OS: Pixel 7 / panther_beta-user 14 AP31.240322.023 release-keys
Brave build: 1.67.70 
Chromium: 125.0.6422.26 (Official Build) canary (64-bit) 

1 2 3
1 2 3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/rewards OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include
Projects
Status: New Issues
Development

No branches or pull requests

4 participants