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

Commit

Permalink
saveAppState needs to be called to perform the quit (was only missing…
Browse files Browse the repository at this point in the history
… from the lastWindowClosed case)

Unintentionally introduced with 40abc0c#diff-2018087f584c4398b5c3a23fc0e5f9dbR172
True needs to be passed, because the last known window state is saved, so receivedAllWindows is always false

Fixes #8629

Auditors: @bridiver
  • Loading branch information
bsclifton committed May 11, 2017
1 parent 42b6293 commit 869c0aa
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions app/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -181,6 +181,7 @@ const initiateSessionStateSave = () => {
// quit triggered by window-all-closed should save last window state
if (lastWindowClosed && lastWindowState) {
perWindowState.push(lastWindowState)
saveAppState(true)
} else if (BrowserWindow.getAllWindows().length > 0) {
++requestId
BrowserWindow.getAllWindows().forEach((win) => win.webContents.send(messages.REQUEST_WINDOW_STATE, requestId))
Expand Down

0 comments on commit 869c0aa

Please sign in to comment.