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

Tor window continues to download even if you exit #5367

Closed
srirambv opened this issue Jul 24, 2019 · 7 comments
Closed

Tor window continues to download even if you exit #5367

srirambv opened this issue Jul 24, 2019 · 7 comments

Comments

@srirambv
Copy link
Contributor

Description

Tor window continues to download even if you exit

Steps to Reproduce

  1. Open Tor window and start downloading a large file
  2. Close the Tor window
  3. A warning shows up asking to Continue download or Exit
  4. Click on Exit and close the Tor window
  5. Launch a new Tor window and close it, you get the same download warning message
  6. Open downloads page you can see the download is in progress
  7. Close the main browser window, shows the same download warning message even though download was triggered in Tor window

Actual result:

Tor window continues to download even if you exit

Expected result:

Download should be cancelled if Tor window is closed and not wait for the main browser window to be closed to cancel

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.66.101 Chromium: 75.0.3770.142 (Official Build) (64-bit)
Revision 8d93477bd18154c67a70a67019cb8032382c55dd-refs/branch-heads/3770@{#1160}
OS Linux

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 dev 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? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

cc: @tomlowenthal @riastradh-brave

@kjozwiak
Copy link
Member

Reproduced on macOS 10.14.5 x64 using the following builds:

Brave 0.66.101 Chromium: 75.0.3770.142 (Official Build) (64-bit)
Revision 8d93477bd18154c67a70a67019cb8032382c55dd-refs/branch-heads/3770@{#1160}
OS Mac OS X
Brave 0.69.77 Chromium: 76.0.3809.62 (Official Build) nightly (64-bit)
Revision 7b77856b3aa34d72f246d12340fc1ded8b2c0e83-refs/branch-heads/3809@{#798}
OS Mac OS X

@tildelowengrimm tildelowengrimm added the priority/P3 The next thing for us to work on. It'll ride the trains. label Jul 25, 2019
@diracdeltas
Copy link
Member

is this the same for non-tor private windows?

@tildelowengrimm tildelowengrimm added the needs-investigation A bug not 100% confirmed/fixed label Jul 30, 2019
@riastradh-brave
Copy link
Contributor

Also, is it the same for guest windows?

@srirambv
Copy link
Contributor Author

srirambv commented Aug 1, 2019

@diracdeltas It doesn't reproduce on Private window. Exit cancels the download and is not shown in downloads page

@riastradh-brave yes reproduces on Guest windows. Exit doesn't cancel the download. Opening a new Guest window and closing it prompts the warning again to continue to download or exit.

@tildelowengrimm
Copy link
Contributor

Sounds like this is a problem with Guest windows and it'll be fixed when we implement #2307.

@yrliou
Copy link
Member

yrliou commented Oct 9, 2019

I just tried it in Nightly, appears to be fixed as a part of fixing #2307 .

@yrliou yrliou closed this as completed Oct 9, 2019
@yrliou yrliou added this to the 0.72.x - Nightly milestone Oct 9, 2019
@btlechowski
Copy link

btlechowski commented Oct 28, 2019

Verification passed on

Brave 0.72.110 Chromium: 78.0.3904.70 (Official Build) dev (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS

Verified test plan from the description.
Verified that the file can be fully downloaded.
Verified that the download in canceled after exiting tor window.
Verified that the the file being downloaded is not listed in brave://downloads after reopening tor window.
Verified that the file being downloaded is deleted after exiting tor window

Verification passed on

Brave 0.72.112 Chromium: 78.0.3904.70 (Official Build) dev (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Verified test plan from the description.
Verified that the file can be fully downloaded.
Verified that the download is canceled after exiting the tor window.
Verified that the file being downloaded is not listed in brave://downloads after reopening the tor window.
Verified that the file being downloaded is deleted after exiting tor window

Verified passed with

Brave 1.1.2 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS macOS Version 10.13.6 (Build 17G5019)

Verified test plan from the description.
Verified that the file can be fully downloaded.
Verified that the download is canceled after exiting the tor window.
Verified that the file being downloaded is not listed in brave://downloads after reopening the tor window.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants