From c8ff67ab7538e4b73fcd0c74c61cdb13491cb691 Mon Sep 17 00:00:00 2001 From: Kevin Sawicki Date: Thu, 1 Dec 2016 14:37:03 -0800 Subject: [PATCH] Use spread syntax instead of function apply --- lib/browser/api/app.js | 2 +- lib/browser/api/browser-window.js | 12 ++++++------ lib/browser/api/dialog.js | 8 ++++---- lib/browser/api/navigation-controller.js | 6 ++---- lib/browser/api/net.js | 8 ++++---- lib/browser/api/web-contents.js | 2 +- lib/browser/guest-window-manager.js | 6 +++--- lib/browser/init.js | 2 +- lib/common/api/callbacks-registry.js | 6 ++---- lib/renderer/init.js | 4 ++-- lib/renderer/override.js | 6 +++--- lib/sandboxed_renderer/init.js | 2 +- 12 files changed, 30 insertions(+), 34 deletions(-) diff --git a/lib/browser/api/app.js b/lib/browser/api/app.js index 167327c40b29..c5865f2242b8 100644 --- a/lib/browser/api/app.js +++ b/lib/browser/api/app.js @@ -81,7 +81,7 @@ app.allowNTLMCredentialsForAllDomains = function (allow) { const events = ['login', 'certificate-error', 'select-client-certificate'] for (let name of events) { app.on(name, (event, webContents, ...args) => { - webContents.emit.apply(webContents, [name, event].concat(args)) + webContents.emit(name, event, ...args) }) } diff --git a/lib/browser/api/browser-window.js b/lib/browser/api/browser-window.js index 32a9f0f426ae..639e54e25173 100644 --- a/lib/browser/api/browser-window.js +++ b/lib/browser/api/browser-window.js @@ -152,19 +152,19 @@ BrowserWindow.fromDevToolsWebContents = (webContents) => { // Helpers. Object.assign(BrowserWindow.prototype, { loadURL (...args) { - return this.webContents.loadURL.apply(this.webContents, args) + return this.webContents.loadURL(...args) }, getURL (...args) { return this.webContents.getURL() }, reload (...args) { - return this.webContents.reload.apply(this.webContents, args) + return this.webContents.reload(...args) }, send (...args) { - return this.webContents.send.apply(this.webContents, args) + return this.webContents.send(...args) }, openDevTools (...args) { - return this.webContents.openDevTools.apply(this.webContents, args) + return this.webContents.openDevTools(...args) }, closeDevTools () { return this.webContents.closeDevTools() @@ -179,7 +179,7 @@ Object.assign(BrowserWindow.prototype, { return this.webContents.toggleDevTools() }, inspectElement (...args) { - return this.webContents.inspectElement.apply(this.webContents, args) + return this.webContents.inspectElement(...args) }, inspectServiceWorker () { return this.webContents.inspectServiceWorker() @@ -188,7 +188,7 @@ Object.assign(BrowserWindow.prototype, { return this.webContents.showDefinitionForSelection() }, capturePage (...args) { - return this.webContents.capturePage.apply(this.webContents, args) + return this.webContents.capturePage(...args) } }) diff --git a/lib/browser/api/dialog.js b/lib/browser/api/dialog.js index 766d89cf8b03..58800a390ce4 100644 --- a/lib/browser/api/dialog.js +++ b/lib/browser/api/dialog.js @@ -50,7 +50,7 @@ module.exports = { showOpenDialog: function (...args) { var prop, properties, value, wrappedCallback checkAppInitialized() - let [window, options, callback] = parseArgs.apply(null, args) + let [window, options, callback] = parseArgs(...args) if (options == null) { options = { title: 'Open', @@ -97,7 +97,7 @@ module.exports = { showSaveDialog: function (...args) { var wrappedCallback checkAppInitialized() - let [window, options, callback] = parseArgs.apply(null, args) + let [window, options, callback] = parseArgs(...args) if (options == null) { options = { title: 'Save' @@ -130,7 +130,7 @@ module.exports = { showMessageBox: function (...args) { var flags, i, j, len, messageBoxType, ref2, ref3, text checkAppInitialized() - let [window, options, callback] = parseArgs.apply(null, args) + let [window, options, callback] = parseArgs(...args) if (options == null) { options = { type: 'none' @@ -185,7 +185,7 @@ module.exports = { }, showErrorBox: function (...args) { - return binding.showErrorBox.apply(binding, args) + return binding.showErrorBox(...args) } } diff --git a/lib/browser/api/navigation-controller.js b/lib/browser/api/navigation-controller.js index 33ce779521fa..0bccf0c6dcc5 100644 --- a/lib/browser/api/navigation-controller.js +++ b/lib/browser/api/navigation-controller.js @@ -4,13 +4,11 @@ const {ipcMain} = require('electron') // The history operation in renderer is redirected to browser. ipcMain.on('ELECTRON_NAVIGATION_CONTROLLER', function (event, method, ...args) { - var ref - (ref = event.sender)[method].apply(ref, args) + event.sender[method](...args) }) ipcMain.on('ELECTRON_SYNC_NAVIGATION_CONTROLLER', function (event, method, ...args) { - var ref - event.returnValue = (ref = event.sender)[method].apply(ref, args) + event.returnValue = event.sender[method](...args) }) // JavaScript implementation of Chromium's NavigationController. diff --git a/lib/browser/api/net.js b/lib/browser/api/net.js index e128bc3b0514..10d903919e9d 100644 --- a/lib/browser/api/net.js +++ b/lib/browser/api/net.js @@ -83,20 +83,20 @@ class IncomingMessage extends Readable { URLRequest.prototype._emitRequestEvent = function (isAsync, ...rest) { if (isAsync) { process.nextTick(() => { - this.clientRequest.emit.apply(this.clientRequest, rest) + this.clientRequest.emit(...rest) }) } else { - this.clientRequest.emit.apply(this.clientRequest, rest) + this.clientRequest.emit(...rest) } } URLRequest.prototype._emitResponseEvent = function (isAsync, ...rest) { if (isAsync) { process.nextTick(() => { - this._response.emit.apply(this._response, rest) + this._response.emit(...rest) }) } else { - this._response.emit.apply(this._response, rest) + this._response.emit(...rest) } } diff --git a/lib/browser/api/web-contents.js b/lib/browser/api/web-contents.js index a02fddb7d0a2..ca702b1257d2 100644 --- a/lib/browser/api/web-contents.js +++ b/lib/browser/api/web-contents.js @@ -246,7 +246,7 @@ WebContents.prototype._init = function () { // Delays the page-title-updated event to next tick. this.on('-page-title-updated', function (...args) { setImmediate(() => { - this.emit.apply(this, ['page-title-updated'].concat(args)) + this.emit('page-title-updated', ...args) }) }) diff --git a/lib/browser/guest-window-manager.js b/lib/browser/guest-window-manager.js index aee98fc41f4d..8212e5094aad 100644 --- a/lib/browser/guest-window-manager.js +++ b/lib/browser/guest-window-manager.js @@ -211,7 +211,7 @@ ipcMain.on('ELECTRON_GUEST_WINDOW_MANAGER_WINDOW_METHOD', function (event, guest const guestWindow = getGuestWindow(guestContents) if (guestWindow != null) { - event.returnValue = guestWindow[method].apply(guestWindow, args) + event.returnValue = guestWindow[method](...args) } else { event.returnValue = null } @@ -235,7 +235,7 @@ ipcMain.on('ELECTRON_GUEST_WINDOW_MANAGER_WEB_CONTENTS_METHOD', function (event, if (guestContents == null) return if (canAccessWindow(event.sender, guestContents)) { - guestContents[method].apply(guestContents, args) + guestContents[method](...args) } else { console.error(`Blocked ${event.sender.getURL()} from calling ${method} on its opener.`) } @@ -249,7 +249,7 @@ ipcMain.on('ELECTRON_GUEST_WINDOW_MANAGER_WEB_CONTENTS_METHOD_SYNC', function (e } if (canAccessWindow(event.sender, guestContents)) { - event.returnValue = guestContents[method].apply(guestContents, args) + event.returnValue = guestContents[method](...args) } else { console.error(`Blocked ${event.sender.getURL()} from calling ${method} on its opener.`) event.returnValue = null diff --git a/lib/browser/init.js b/lib/browser/init.js index 704ef7037d1c..5e4fbf28199a 100644 --- a/lib/browser/init.js +++ b/lib/browser/init.js @@ -26,7 +26,7 @@ if (process.platform === 'win32') { // Redirect node's console to use our own implementations, since node can not // handle console output when running as GUI program. var consoleLog = function (...args) { - return process.log(util.format.apply(util, args) + '\n') + return process.log(util.format(...args) + '\n') } var streamWrite = function (chunk, encoding, callback) { if (Buffer.isBuffer(chunk)) { diff --git a/lib/common/api/callbacks-registry.js b/lib/common/api/callbacks-registry.js index 3e2d9778c091..3194ca8236e3 100644 --- a/lib/common/api/callbacks-registry.js +++ b/lib/common/api/callbacks-registry.js @@ -45,13 +45,11 @@ class CallbacksRegistry { } call (id, ...args) { - var ref - return (ref = this.get(id)).call.apply(ref, [global].concat(args)) + return this.get(id).call(global, ...args) } apply (id, ...args) { - var ref - return (ref = this.get(id)).apply.apply(ref, [global].concat(args)) + return this.get(id).apply(global, ...args) } remove (id) { diff --git a/lib/renderer/init.js b/lib/renderer/init.js index 05733f678a80..02c71b5fa028 100644 --- a/lib/renderer/init.js +++ b/lib/renderer/init.js @@ -29,11 +29,11 @@ const electron = require('electron') // Call webFrame method. electron.ipcRenderer.on('ELECTRON_INTERNAL_RENDERER_WEB_FRAME_METHOD', (event, method, args) => { - electron.webFrame[method].apply(electron.webFrame, args) + electron.webFrame[method](...args) }) electron.ipcRenderer.on('ELECTRON_INTERNAL_RENDERER_SYNC_WEB_FRAME_METHOD', (event, requestId, method, args) => { - const result = electron.webFrame[method].apply(electron.webFrame, args) + const result = electron.webFrame[method](...args) event.sender.send(`ELECTRON_INTERNAL_BROWSER_SYNC_WEB_FRAME_RESPONSE_${requestId}`, result) }) diff --git a/lib/renderer/override.js b/lib/renderer/override.js index 69ef3a5e6f9a..6a8a50c378e8 100644 --- a/lib/renderer/override.js +++ b/lib/renderer/override.js @@ -74,7 +74,7 @@ var BrowserWindowProxy = (function () { } BrowserWindowProxy.prototype['eval'] = function (...args) { - ipcRenderer.send.apply(ipcRenderer, ['ELECTRON_GUEST_WINDOW_MANAGER_WEB_CONTENTS_METHOD', this.guestId, 'executeJavaScript'].concat(args)) + ipcRenderer.send('ELECTRON_GUEST_WINDOW_MANAGER_WEB_CONTENTS_METHOD', this.guestId, 'executeJavaScript', ...args) } return BrowserWindowProxy @@ -207,11 +207,11 @@ ipcRenderer.on('ELECTRON_GUEST_WINDOW_POSTMESSAGE', function (event, sourceId, m // Forward history operations to browser. var sendHistoryOperation = function (...args) { - ipcRenderer.send.apply(ipcRenderer, ['ELECTRON_NAVIGATION_CONTROLLER'].concat(args)) + ipcRenderer.send('ELECTRON_NAVIGATION_CONTROLLER', ...args) } var getHistoryOperation = function (...args) { - return ipcRenderer.sendSync.apply(ipcRenderer, ['ELECTRON_SYNC_NAVIGATION_CONTROLLER'].concat(args)) + return ipcRenderer.sendSync('ELECTRON_SYNC_NAVIGATION_CONTROLLER', ...args) } window.history.back = function () { diff --git a/lib/sandboxed_renderer/init.js b/lib/sandboxed_renderer/init.js index bc0855527f0e..68daf476ca5d 100644 --- a/lib/sandboxed_renderer/init.js +++ b/lib/sandboxed_renderer/init.js @@ -14,7 +14,7 @@ const geval = eval require('../renderer/api/ipc-renderer-setup')(ipcRenderer, binding) binding.onMessage = function (channel, args) { - ipcRenderer.emit.apply(ipcRenderer, [channel].concat(args)) + ipcRenderer.emit(channel, ...args) } binding.onExit = function () {