INT-1434 app launched event (#389)

* INT-1434 fix window:renderer-ready and app:launched events

* Only set debug namespaces on app launch

This allows to change debug namespaces during app runtime via dev console.

Author: rueckstiess

src/app/index.js

@@ -1,11 +1,4 @@
 /* eslint no-console:0 */
-if (process.env.NODE_ENV !== 'production') {
-  require('debug').enable('mon*,had*');
-  require('debug/browser');
-  require('debug').enable('mon*,had*');
-}
-
-var debug = require('debug')('mongodb-compass:app');
 
 console.time('app/index.js');
 
@@ -60,6 +53,16 @@ var AutoUpdate = require('../auto-update');
 
 var addInspectElementMenu = require('debug-menu').install;
 
+ipc.once('app:launched', function() {
+  console.log('in app:launched');
+  if (process.env.NODE_ENV !== 'production') {
+    require('debug').enable('mon*,had*');
+    require('debug/browser');
+  }
+});
+
+var debug = require('debug')('mongodb-compass:app');
+
 function getConnection(model, done) {
   function _fetch(fn) {
     model.fetch({
@@ -409,7 +412,7 @@ app.extend({
       metricsSetup();
 
       // signal to main process that app is ready
-      ipc.call('app:renderer-ready');
+      ipc.call('window:renderer-ready');
 
       // as soon as dom is ready, render and set up the rest
       self.onDomReady();

src/main/window-manager.js

@@ -207,6 +207,22 @@ function hideShareSubmenu() {
   AppMenu.hideShare();
 }
 
+/**
+ * can't use webContents `did-finish-load` event here because
+ * metrics aren't set up at that point. renderer app sends custom event
+ * `window:renderer-ready` when metrics are set up. If first app launch,
+ * send back `app:launched` message at that point.
+ *
+ * @param {Object} sender   original sender of the event
+ */
+function rendererReady(sender) {
+  if (!appLaunched) {
+    appLaunched = true;
+    debug('sending `app:launched` msg back');
+    sender.send('app:launched');
+  }
+}
+
 // respond to events from the renderer process
 ipc.respondTo({
   'app:show-connect-window': showConnectWindow,
@@ -215,28 +231,15 @@ ipc.respondTo({
   'window:show-about-dialog': showAboutDialog,
   'window:show-share-submenu': showShareSubmenu,
   'window:hide-share-submenu': hideShareSubmenu,
-  'window:show-compass-overview-submenu': showCompassOverview
+  'window:show-compass-overview-submenu': showCompassOverview,
+  'window:renderer-ready': rendererReady
 });
 
 // respond to events from the main process
 app.on('window:show-about-dialog', showAboutDialog);
 app.on('app:show-connect-window', showConnectWindow);
 app.on('app:show-help-window', showHelpWindow);
 
-/**
- * can't use webContents `did-finish-load` event here because
- * metrics aren't set up at that point. renderer app sends custom event
- * `renderer ready` when metrics are set up. If first app launch, send back
- * `app launched` message at that point.
- */
-app.on('window:renderer-ready', function(arg, event) {
-  if (!appLaunched) {
-    appLaunched = true;
-    debug('sending `app:launched` msg back');
-    event.sender.send('app:launched');
-  }
-});
-
 app.on('before-quit', function() {
   debug('sending `app:quit` msg');
   _.first(BrowserWindow.getAllWindows()).webContents.send('app:quit');