Use spread syntax instead of apply
authorKevin Sawicki <kevinsawicki@gmail.com>
Thu, 1 Dec 2016 22:43:30 +0000 (14:43 -0800)
committerKevin Sawicki <kevinsawicki@gmail.com>
Thu, 1 Dec 2016 22:56:00 +0000 (14:56 -0800)
lib/browser/api/ipc-main.js
lib/renderer/api/ipc-renderer-setup.js

index 0c2592f..2c2978e 100644 (file)
@@ -1,14 +1,16 @@
 const EventEmitter = require('events').EventEmitter
 
-module.exports = new EventEmitter()
+const emitter = new EventEmitter()
 
-const removeAllListeners = module.exports.removeAllListeners
-module.exports.removeAllListeners = function (...args) {
+const removeAllListeners = emitter.removeAllListeners.bind(emitter)
+emitter.removeAllListeners = function (...args) {
   if (args.length === 0) {
     throw new Error('Removing all listeners from ipcMain will make Electron internals stop working.  Please specify a event name')
   }
-  removeAllListeners.apply(this, args)
+  removeAllListeners(...args)
 }
 
 // Do not throw exception when channel name is "error".
-module.exports.on('error', () => {})
+emitter.on('error', () => {})
+
+module.exports = emitter
index 2cebc6a..c899452 100644 (file)
@@ -30,11 +30,11 @@ module.exports = function (ipcRenderer, binding) {
     ipcRenderer.send('ELECTRON_BROWSER_SEND_TO', true, webContentsId, channel, ...args)
   }
 
-  const removeAllListeners = ipcRenderer.removeAllListeners
+  const removeAllListeners = ipcRenderer.removeAllListeners.bind(ipcRenderer)
   ipcRenderer.removeAllListeners = function (...args) {
     if (args.length === 0) {
       throw new Error('Removing all listeners from ipcRenderer will make Electron internals stop working.  Please specify a event name')
     }
-    removeAllListeners.apply(this, args)
+    removeAllListeners(...args)
   }
 }