shortcuts.js: Consistent exported method naming

Exposed methods whose sole purpose is notify the module of an event
firing, should start with "on".

main/ipc.js

@@ -82,12 +82,12 @@ function init () {
 
   ipcMain.on('onPlayerOpen', function () {
     menu.onPlayerOpen()
-    shortcuts.registerPlayerShortcuts()
+    shortcuts.onPlayerOpen()
   })
 
   ipcMain.on('onPlayerClose', function () {
     menu.onPlayerClose()
-    shortcuts.unregisterPlayerShortcuts()
+    shortcuts.onPlayerOpen()
   })
 
   ipcMain.on('focusWindow', function (e, windowName) {

main/menu.js

@@ -5,6 +5,8 @@ module.exports = {
   onToggleFullScreen,
   onWindowHide,
   onWindowShow,
+
+  // TODO: move these out of menu.js -- they don't belong here
   showOpenSeedFiles,
   showOpenTorrentAddress,
   showOpenTorrentFile,

main/shortcuts.js

@@ -1,7 +1,7 @@
 module.exports = {
   init,
-  registerPlayerShortcuts,
-  unregisterPlayerShortcuts
+  onPlayerClose,
+  onPlayerOpen
 }
 
 var electron = require('electron')
@@ -19,11 +19,13 @@ function init () {
   localShortcut.register('CmdOrCtrl+Shift+F', menu.toggleFullScreen)
 }
 
-function registerPlayerShortcuts () {
-  // Special "media key" for play/pause, available on some keyboards
-  globalShortcut.register('MediaPlayPause', () => windows.main.send('dispatch', 'playPause'))
+function onPlayerOpen () {
+  // Register special "media key" for play/pause, available on some keyboards
+  globalShortcut.register('MediaPlayPause', function () {
+    windows.main.send('dispatch', 'playPause')
+  })
 }
 
-function unregisterPlayerShortcuts () {
+function onPlayerClose () {
   globalShortcut.unregister('MediaPlayPause')
 }