Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Tab session not restored after restart #13278

Closed
btlechowski opened this issue Feb 23, 2018 · 2 comments
Closed

Tab session not restored after restart #13278

btlechowski opened this issue Feb 23, 2018 · 2 comments

Comments

@btlechowski
Copy link
Contributor

btlechowski commented Feb 23, 2018

Test plan / Steps to Reproduce

  1. Clean Install
  2. Open Brave on Windows or Linux
  3. Open youtube.com
  4. Close Brave through titlebar's close button (x)
  5. Open Brave

Actual result:
Tabs are NOT restored
Only New Tab is shown

Expected result:
Tabs are restored

Reproduces how often:
100%

Brave Version

Brave: 0.21.12 
V8: 6.4.388.41 
rev: 246f9008be47dbdcc6dc9ded5690a847eee8a4e0 
Muon: 5.0.5 
OS Release: 6.1.7601 
Update Channel: Beta 
OS Architecture: x64 
OS Platform: Microsoft Windows 
Node.js: 7.9.0 
Brave Sync: v1.4.2 
libchromiumcontent: 64.0.3282.140

Additional Information

The Brave was run through the shortcut
Very likely related to #13233, cc @petemill
Cannot reproduce when closing through CTRL+Q

@btlechowski btlechowski added this to the 0.21.x (Beta Channel) milestone Feb 23, 2018
@kjozwiak kjozwiak added the 0.21.x issue first seen in 0.21.x label Feb 23, 2018
@kjozwiak
Copy link
Member

kjozwiak commented Feb 23, 2018

I also managed to reproduce this on the following platforms:

  • Win 10 x64 VM - Reproduced
  • Ubuntu 17.10 x64 VM - Reproduced

Looks like macOS isn't affected. When you close the only remaining browser window on macOS using X, the application isn't completely terminated/killed like it is under Windows and Linux. Even without any brave windows, Brave will continue running until it's completely closed via Exit Brave.

When using restarting Brave after using Exit Brave, tabs are correctly being restored.

@srirambv also reproduced this on his Win 10 desktop. Great catch @btlechowski 👍

@petemill petemill self-assigned this Feb 23, 2018
petemill added a commit that referenced this issue Feb 24, 2018
…emembered window to restore on next startup (windows and linux)

Fix #13278
@bsclifton bsclifton added 0.22.x issue first seen in 0.22.x and removed 0.21.x issue first seen in 0.21.x labels Feb 27, 2018
bsclifton pushed a commit that referenced this issue Mar 1, 2018
…emembered window to restore on next startup (windows and linux)

Fix #13278
@srirambv
Copy link
Collaborator

srirambv commented Mar 22, 2018

Verified on Windows x64

  • 0.22.6 e6ff4ea
  • libchromiumcontent: 65.0.3325.162
  • muon: 5.1.0

Verified on Ubuntu 10.10 x64

  • 0.22.7 8bb7e77
  • libchromiumcontent: 65.0.3325.181
  • muon: 5.1.1

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