From 4e07ecf05ca119bef9febe4a6c2230432b456d96 Mon Sep 17 00:00:00 2001
From: Nate Goldman <nate@ngoldman.me>
Date: Sun, 6 Mar 2016 23:06:30 -0800
Subject: [PATCH] load state first

---
 renderer/index.js | 60 +++++++++++++++++++++++++++--------------------
 1 file changed, 35 insertions(+), 25 deletions(-)

diff --git a/renderer/index.js b/renderer/index.js
index 873280ba..a234d034 100644
--- a/renderer/index.js
+++ b/renderer/index.js
@@ -9,7 +9,6 @@ var EventEmitter = require('events')
 var mainLoop = require('main-loop')
 var networkAddress = require('network-address')
 var path = require('path')
-var state = require('./state')
 var torrentPoster = require('./lib/torrent-poster')
 var WebTorrent = require('webtorrent')
 var cfg = require('application-config')('WebTorrent')
@@ -20,6 +19,9 @@ var patch = require('virtual-dom/patch')
 
 var App = require('./views/app')
 
+// For easy debugging in Developer Tools
+var state = global.state = require('./state')
+
 // Force use of webtorrent trackers on all torrents
 global.WEBTORRENT_ANNOUNCE = createTorrent.announceList
   .map(function (arr) {
@@ -31,6 +33,10 @@ global.WEBTORRENT_ANNOUNCE = createTorrent.announceList
 
 var vdomLoop
 
+// All state lives in state.js. `state.saved` is read from and written to a file.
+// All other state is ephemeral. First we load state.saved then initialize the app.
+loadState(init)
+
 /**
  * Called once when the application loads. (Not once per window.)
  * Connects to the torrent networks, sets up the UI and OS integrations like
@@ -59,13 +65,12 @@ function init () {
   // (eg % downloaded) and to keep the cursor in sync when playing a video
   setInterval(update, 1000)
 
-  // All state lives in state.js. `state.saved` is read from and written to a
-  // file. All other state is ephemeral. Here we'll load state.saved:
-  loadState()
+  // Resume all saved torrents now that state is loaded and vdom is ready
+  resumeAllTorrents()
   document.addEventListener('unload', saveState)
 
-  // For easy debugging in Developer Tools
-  global.state = state
+  // listen for messages from the main process
+  setupIpc()
 
   // OS integrations:
   // ...Chromecast and Airplay
@@ -108,7 +113,6 @@ function init () {
     state.isFocused = false
   })
 }
-init()
 
 // This is the (mostly) pure funtion from state -> UI. Returns a virtual DOM
 // tree. Any events, such as button clicks, will turn into calls to dispatch()
@@ -178,37 +182,43 @@ function dispatch (action, ...args) {
   }
 }
 
-electron.ipcRenderer.on('addTorrent', function (e, torrentId) {
-  dispatch('addTorrent', torrentId)
-})
+function setupIpc () {
+  electron.ipcRenderer.on('addTorrent', function (e, torrentId) {
+    dispatch('addTorrent', torrentId)
+  })
 
-electron.ipcRenderer.on('seed', function (e, files) {
-  dispatch('seed', files)
-})
+  electron.ipcRenderer.on('seed', function (e, files) {
+    dispatch('seed', files)
+  })
 
-electron.ipcRenderer.on('fullscreenChanged', function (e, isFullScreen) {
-  state.isFullScreen = isFullScreen
-  update()
-})
+  electron.ipcRenderer.on('fullscreenChanged', function (e, isFullScreen) {
+    state.isFullScreen = isFullScreen
+    update()
+  })
 
-electron.ipcRenderer.on('addFakeDevice', function (e, device) {
-  var player = new EventEmitter()
-  player.play = (networkURL) => console.log(networkURL)
-  state.devices[device] = player
-  update()
-})
+  electron.ipcRenderer.on('addFakeDevice', function (e, device) {
+    var player = new EventEmitter()
+    player.play = (networkURL) => console.log(networkURL)
+    state.devices[device] = player
+    update()
+  })
+}
 
 // Load state.saved from the JSON state file
-function loadState () {
+function loadState (callback) {
   cfg.read(function (err, data) {
     if (err) console.error(err)
     electron.ipcRenderer.send('log', 'loaded state from ' + cfg.filePath)
     state.saved = data
     if (!state.saved.torrents) state.saved.torrents = []
-    state.saved.torrents.forEach((x) => startTorrenting(x.infoHash))
+    if (callback) callback()
   })
 }
 
+function resumeAllTorrents () {
+  state.saved.torrents.forEach((x) => startTorrenting(x.infoHash))
+}
+
 // Write state.saved to the JSON state file
 function saveState () {
   electron.ipcRenderer.send('log', 'saving state to ' + cfg.filePath)