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

Browser crash when Tor/Guest/Private window is opened just after launching browser - follow up to 6476 #6565

Closed
btlechowski opened this issue Oct 21, 2019 · 4 comments

Comments

@btlechowski
Copy link

btlechowski commented Oct 21, 2019

Follow up to #6476

The browser still crashes, but in order to crash, the private window needs to be opened just after browser launch

Steps to Reproduce

  1. Clean profile
  2. Launch Brave
  3. Immediately open Tor/Guest/Private window
  4. Wait 10 seconds

Actual result:

Crash

6565

Note: Have not experienced the crash if waited for 10 seconds after launching Brave
Note: I was not able to generate crash dmps, nor crash ids
Note: Crash reproduced on en_US locale

Expected result:

No crash

Reproduces how often:

Always

Brave version (brave://version info)

Brave 0.70.119 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS Ubuntu 18.04 LTS

cc @brave/legacy_qa @bsclifton @rebron

@kjozwiak
Copy link
Member

kjozwiak commented Oct 21, 2019

Reproduced on `macOS 10.14.6 x64 using the following build:

Brave 0.70.119 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS macOS Version 10.14.6 (Build 18G95)

When following the above STR that @btlechowski provided, Brave creates two crashes:

@bsclifton @rebron we should fix this before releasing 0.70.x. Crashes are never ideal especially when using major functionality like private and Tor windows.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 21, 2019

The issue is still reproducible on Windows10 x64, tested against 0.70.119

@bsclifton bsclifton self-assigned this Oct 21, 2019
@bsclifton
Copy link
Member

Should be fixed with brave/brave-core@e2d9a61

Fixed on 0.71.x with brave/brave-core@04531e9

And 0.72.x forward is fixed with brave/brave-core#3319

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 22, 2019

Verification passed on

Brave 0.70.120 Chromium: 78.0.3904.70 (Official Build) (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified the STR from the description
  • Verified the locales the US, UK, New Zealand, Ireland, Germany, and Austalia. - No crash in PT/TOR/Guest windows
  • Verified both clean and upgraded profile
  • Verified no crash when sites are opened in PT/TOR/Guest windows
  • Verified no crash in PT/TOR/Guest windows, kept each window open for 20 secs and observed there is no crash

Verification passed on

Brave 0.70.120 Chromium: 78.0.3904.70 (Official Build) (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS
  • Verified the STR from the description
  • Verified the locales the US, UK, New Zealand, Ireland, Germany, and Austalia. - No crash in PT/TOR/Guest windows
  • Verified both clean and upgraded profile
  • Verified no crash when sites are opened in PT/TOR/Guest windows

Verified passed with

Brave 0.70.120 Chromium: 78.0.3904.70 (Official Build) (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS macOS Version 10.13.6 (Build 17G5019)
  • Verified the STR from the description
  • Verified the locales: Germany, Australia, New Zealand, Ireland, US. - No crash in PT/TOR/Guest windows
  • Verified both clean and upgraded profile
  • Verified no crash when sites are opened in PT/TOR/Guest windows
  • Verified no crash in PT/TOR/Guest windows, kept each window open for 20 secs and observed there is no crash

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