Skip to content

download icons quickly flash yellow after keeping insecure download - follow up of #45159 #45269

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

Open
3 of 6 tasks
kjozwiak opened this issue Apr 7, 2025 · 0 comments
Open
3 of 6 tasks

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Apr 7, 2025

Description

The following issue is a follow up of #45159.

When downloading an insecure file, you'll notice that when clicking on keep, the download icon will quickly flash yellow due to the ! icon and then continue to use the expected colours while the file is being downloaded. If the file has already been downloaded/cached and just requires a decision re: keeping/discarding, the download icon will also quickly flash yellow before being saved on the HD/locally.

Steps to reproduce

Test Case #1 - Saving cached insecure download

  1. install/launch 1.79.37 Chromium: 135.0.7049.52 (or any build that has the issue)
  2. download an insecure file as mentioned via [cr135 follow up] Regression of download indicator for insecure downloads. #45159 (comment)
  3. once you receive the insecure icon, wait ~3-5min for the download to complete/get cached (waiting on decision re: keeping or discarding)
  4. click on Keep and you'll notice the download icon quickly flash yellow before saving the file locally/to the HD

Test Case #2 - Keeping/Complete insecure download

  1. install/launch 1.79.37 Chromium: 135.0.7049.52 (or any build that has the issue)
  2. download an insecure file as mentioned via [cr135 follow up] Regression of download indicator for insecure downloads. #45159 (comment)
  3. once you receive the insecure icon, click on Keep
  4. you'll notice the download icon quickly flash yellow before the download resumes/completes using the regular download colours

Actual result

Example #1 following Test Case #1 above:

Recording.2025-04-07.163741.mp4

Example #2 following Test Case #2 above:

Recording.2025-04-07.163956.mp4

Expected result

We shouldn't be displaying any yellow when users click on Keep when saving an insecure file.

Reproduces how often

Easily reproduced

Brave version (brave://version info)

Brave | 1.79.37 Chromium: 135.0.7049.52 (Official Build) nightly (64-bit)
-- | --
Revision | a3ad03a075e376dcad5b5976ef142fc608173d63
OS | Windows 11 Version 24H2 (Build 26100.3476)

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

  • with Brave Shields disabled
  • with Brave Rewards disabled
  • in the latest version of Chrome

Miscellaneous information

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

1 participant