Perf: Use electron.* getter inline, rather than upfront

This commit is contained in:
Feross Aboukhadijeh
2016-05-11 22:12:37 +02:00
parent 39195fe8c4
commit b1a7543d37
3 changed files with 14 additions and 18 deletions

View File

@@ -5,16 +5,15 @@ module.exports = {
}
var electron = require('electron')
var localShortcut = require('electron-localshortcut')
var globalShortcut = electron.globalShortcut
var menu = require('./menu')
var windows = require('./windows')
function init () {
// Alternate shortcuts. Most shortcuts are registered in menu,js, but Electron does not
// support multiple shortcuts for a single menu item.
var localShortcut = require('electron-localshortcut')
// Alternate shortcuts. Most shortcuts are registered in menu,js, but Electron
// does not support multiple shortcuts for a single menu item.
localShortcut.register('CmdOrCtrl+Shift+F', menu.toggleFullScreen)
localShortcut.register('Space', () => windows.main.send('dispatch', 'playPause'))
@@ -24,12 +23,12 @@ function init () {
function onPlayerOpen () {
// Register special "media key" for play/pause, available on some keyboards
globalShortcut.register(
electron.globalShortcut.register(
'MediaPlayPause',
() => windows.main.send('dispatch', 'playPause')
)
}
function onPlayerClose () {
globalShortcut.unregister('MediaPlayPause')
electron.globalShortcut.unregister('MediaPlayPause')
}