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

NTP always shows the same background image #32359

Closed
A-t-s-u-s-h-i opened this issue Aug 18, 2023 · 3 comments · Fixed by brave/brave-core#19896
Closed

NTP always shows the same background image #32359

A-t-s-u-s-h-i opened this issue Aug 18, 2023 · 3 comments · Fixed by brave/brave-core#19896
Assignees
Labels

Comments

@A-t-s-u-s-h-i
Copy link

Description

https://community.brave.com/t/topic/500776

NTP shows same image no matter how many times they reload NTP.

Steps to Reproduce

  1. launch Brave(desktop)
  2. Show NTP
  3. Repeat reloading

Actual result:

The same background image shows
image

Expected result:

Shows other background images randomly

Reproduces how often:

Easily reproduced(only on specific devices)

Brave version (brave://version info)

v1.57.47
Windows10

@rebron rebron added feature/new-tab needs-investigation A bug not 100% confirmed/fixed priority/P2 A bad problem. We might uplift this to the next planned release. priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Aug 19, 2023
@A-t-s-u-s-h-i
Copy link
Author

Now same issue happens on my device. The issue happens only on a specific browser profile. NTP changes randomly on other profiles.

Brave 1.57.53
NTP Background Images - version: 1.0.11
Sponsored images enabled

2023-08-24.11.32.55.mov

simonhong added a commit to brave/brave-core that referenced this issue Aug 28, 2023
fix brave/brave-browser#32359

Sometimes we could meet the condition that
ViewCounterModel::count_to_branded_wallpaper_ is zero and no SI campaign.
When this happens, we didn't increase background wallpaper index.
This caused showing same background image till new campaign starts.
We should increase background images index in this situation also.
@simonhong simonhong added QA/Yes release-notes/include and removed needs-investigation A bug not 100% confirmed/fixed labels Aug 28, 2023
@brave-builds brave-builds added this to the 1.59.x - Nightly milestone Aug 28, 2023
@rebron rebron changed the title NTP always shows same background image NTP always shows the same background image Sep 6, 2023
@kjozwiak
Copy link
Member

kjozwiak commented Sep 8, 2023

The above requires 1.58.119 or higher for 1.58.x verification 👍

@LaurenWags
Copy link
Member

LaurenWags commented Sep 11, 2023

Verified with

Brave | 1.58.120 Chromium: 117.0.5938.48 (Official Build) (x86_64)
-- | --
Revision | 8b2e2785c90c076fc1413acb2c003e603eac02d3
OS | macOS Version 13.5.2 (Build 22G91)

Used "Manual Test" steps from brave/brave-core#19896 (comment) and 1.57.62 Chromium: 116.0.5845.180 to reproduce the issue.

Upgrade Profile - PASSED
  1. Launch browser 1.57.62 and current region has SI campaign (used US)
  2. Open NTP reload multiple times till SI is shown and reload three times.
  3. With this condition, NTP SI will be shown at next NTP reloading or opening
  4. Close all NTP to prevent opening NTP at next startup (Having non-NTP tab is fine before closing window, my non-NTP tab was brave://components)
  5. Shutdown browser and change OS to another region that doesn't have SI campaign (I used Japan)
  6. Launch Browser and make sure there is no NTP tab
  7. Load brave://components (if not already open) in existing tab and update NTP SI components to get empty campaign
  8. Open NTP and reload
  9. Confirmed same NTP background image is shown whenever reloading
  10. Upgraded profile to 1.58.120
  11. Opened NTP and refreshed several times
  12. Confirmed different NTP background images are shown whenever reloading
Step 4 Step 5 Step 7 Step 9 Step 9 Step 9 Step 10 Step 10 Step 12 Step 12 Step 12
4 5 7 9 9a 9b 10 10b 12 12a 12b
Clean Profile - PASSED
  1. Launch browser 1.58.120 and current region has SI campaign (used US)
  2. Open NTP reload multiple times till SI is shown and reload three times.
  3. With this condition, NTP SI will be shown at next NTP reloading or opening
  4. Close all NTP to prevent opening NTP at next startup (Having non-NTP tab is fine before closing window, my non-NTP tab was brave://components)
  5. Shutdown browser and change OS to another region that doesn't have SI campaign (I used Japan)
  6. Launch Browser and make sure there is no NTP tab
  7. Load brave://components (if not already open) in existing tab and update NTP SI components to get empty campaign
  8. Open NTP and reload
  9. Confirmed different NTP background images are shown whenever reloading
Step 4 Step 5 Step 7 Step 9 Step 9 Step 9
4 5 7 9 9a 9b

@LaurenWags LaurenWags added QA Pass-macOS QA/In-Progress Indicates that QA is currently in progress for that particular issue and removed QA Pass-macOS QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants