If crashing within first 2min don't lose windows

Fix brave#10349

Auditors: @bsclifton

This happens because a save would happen on init before the initial window state was present

app/sessionStoreShutdown.js

@@ -188,13 +188,12 @@ app.on('before-quit', (e) => {
   if (sessionStateSaveInterval !== undefined) {
     clearInterval(sessionStateSaveInterval)
   }
-  initiateSessionStateSave()
+  module.exports.initiateSessionStateSave()
 })
 
 const startSessionSaveInterval = () => {
   // save app state every 5 minutes regardless of update frequency
-  initiateSessionStateSave()
-  sessionStateSaveInterval = setInterval(initiateSessionStateSave, appConfig.sessionSaveInterval)
+  sessionStateSaveInterval = setInterval(module.exports.initiateSessionStateSave, appConfig.sessionSaveInterval)
 }
 
 // User initiated exit using File->Quit

test/unit/app/sessionStoreShutdownTest.js

@@ -355,4 +355,18 @@ describe('sessionStoreShutdown unit tests', function () {
       })
     })
   })
+  describe('startSessionSaveInterval', function () {
+    before(function () {
+      this.initiateSessionStateSave = sinon.spy(sessionStoreShutdown, 'initiateSessionStateSave')
+    })
+    after(function () {
+      this.initiateSessionStateSave.restore()
+    })
+    // We only care that initiateSessionStateSave is not called sync.
+    // Windows will be initialized for the non sync case.
+    it('does not call initiateSessionStateSave', function () {
+      sessionStoreShutdown.startSessionSaveInterval()
+      assert.equal(this.initiateSessionStateSave.notCalled, true)
+    })
+  })
 })