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

Private window w/ Tor shows "new private window" #6467

Closed
bsclifton opened this issue Oct 14, 2019 · 6 comments · Fixed by brave/brave-core#3813
Closed

Private window w/ Tor shows "new private window" #6467

bsclifton opened this issue Oct 14, 2019 · 6 comments · Fixed by brave/brave-core#3813

Comments

@bsclifton
Copy link
Member

Description

While testing brave/brave-core#3686, I noticed a weird regression - cc: @yrliou who has done Tor work in Nightly (it may be due to that work? or Chromium 78? not sure)

Basically, in Dev channel for example (C77), when in a private window with Tor, you're not given the option to create a regular private window

  • via the hamburger menu
    Screen Shot 2019-10-14 at 9 27 54 AM
  • via the menu (macOS)
    Screen Shot 2019-10-14 at 9 30 17 AM

On master/Nightly, when in a private window with Tor, you're shown the option to open a new private window. If you do choose this, it opens a private window with Tor, NOT a regular private window.

  • hamburger menu
    Screen Shot 2019-10-14 at 9 28 07 AM
  • via the menu (macOS)
    Screen Shot 2019-10-14 at 9 30 37 AM

Steps to Reproduce

  1. Launch master / Nightly
  2. Open a new private window with Tor
  3. Use hamburger menu, pick New Private Window

Actual result:

A new private window with Tor is shown

Expected result:

Menu option should not be there (see screenshot)

Reproduces how often:

100%

Brave version (brave://version info)

master/Nightly (0.72.x)

Version/Channel Information:

  • Can you reproduce this issue with the current release? n
  • Can you reproduce this issue with the beta channel? n
  • Can you reproduce this issue with the dev channel? n
  • Can you reproduce this issue with the nightly channel? y
@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label Oct 18, 2019
@btlechowski
Copy link

Reproduced on Ubuntu

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

image

cc @brave/legacy_qa

@yrliou yrliou self-assigned this Oct 28, 2019
@yrliou
Copy link
Member

yrliou commented Oct 28, 2019

Had a discussion with @tomlowenthal and @bsclifton through slack, we will keep the New private window in Tor windows but tweak its behavior to open a private window in its parent profile instead of Tor profile.

@btlechowski
Copy link

Because of #6646 if this issue 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.

@yrliou
Copy link
Member

yrliou commented Oct 29, 2019

@btlechowski Right, I think I'll also fix the New window in the same PR.

@yrliou
Copy link
Member

yrliou commented Oct 29, 2019

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

@btlechowski
Copy link

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

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

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)

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