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

New window in private window with tor no longer opens new window with tor #6646

Closed
btlechowski opened this issue Oct 29, 2019 · 3 comments · Fixed by brave/brave-core#3813
Closed

Comments

@btlechowski
Copy link

New window in private window with tor used to open a new window with tor. This is no longer the case.

Steps to Reproduce

  1. Clean install
  2. Open a private window with tor
  3. In private window with tor click New window from Hamburger menu

Actual result:

Normal new window is opened
image

Important: if #6467 is fixed as intended in #6467 (comment), there will be no way of opening a new private window with tor from private window with tor.

Expected result:

New private window with tor is opened
image

Reproduces how often:

Always

Brave version (brave://version info)

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

Version/Channel Information:

  • Can you reproduce this issue with the current release? no
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the dev channel? yes
  • Can you reproduce this issue with the nightly channel?

cc @brave/legacy_qa @bsclifton @rebron @yrliou @tomlowenthal

@yrliou
Copy link
Member

yrliou commented Oct 29, 2019

From slack discussion, our goal is to have the ability to open any type of new windows(regular/private/private with Tor) in Tor window. So I'll fix this issue by showing New Window, New Private Window, and New Private Window with Tor in the hamburger menu and opening the corresponding type of window for each option.

@yrliou
Copy link
Member

yrliou commented Oct 29, 2019

See brave/brave-core#3813 (comment) for the test plan.

@btlechowski
Copy link
Author

btlechowski commented Nov 14, 2019

Verification passed on

Brave 0.72.124 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#3813

regular window
image

private window
image

private window with Tor
image

Verification passed on

Brave 0.72.124 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Normal window:
image
Private window:
image
Private window with TOR:
image

Verified passed with

Brave 0.72.124 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)

Screen Shot 2019-11-14 at 11 42 29 AM

Screen Shot 2019-11-14 at 11 42 48 AM

Screen Shot 2019-11-14 at 11 43 03 AM

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