From fd9c9c3be7247c2c96cde057ae27528addc3ed65 Mon Sep 17 00:00:00 2001
From: Steve Kinney <hello@stevekinney.net>
Date: Sat, 14 May 2016 12:26:53 -0400
Subject: [PATCH] :lipstick: Use spread operator instead of arguments object

---
 lib/browser/api/browser-window.js | 33 ++++++++++++++++---------------
 1 file changed, 17 insertions(+), 16 deletions(-)

diff --git a/lib/browser/api/browser-window.js b/lib/browser/api/browser-window.js
index 43a8b3203bd7..a2b31184dc6b 100644
--- a/lib/browser/api/browser-window.js
+++ b/lib/browser/api/browser-window.js
@@ -50,13 +50,13 @@ BrowserWindow.prototype._init = function () {
     if (!event.defaultPrevented) this.setTitle(title)
   })
 
-  // Sometimes the webContents doesn't get focus when window is shown, so we 
-  // have to force focusing on webContents in this case. The safest way is to 
-  // focus it when we first start to load URL, if we do it earlier it won't 
+  // Sometimes the webContents doesn't get focus when window is shown, so we
+  // have to force focusing on webContents in this case. The safest way is to
+  // focus it when we first start to load URL, if we do it earlier it won't
   // have effect, if we do it later we might move focus in the page.
   //
   // Though this hack is only needed on OS X when the app is launched from
-  // Finder, we still do it on all platforms in case of other bugs we don't 
+  // Finder, we still do it on all platforms in case of other bugs we don't
   // know.
   this.webContents.once('load-url', function () {
     this.focus()
@@ -79,7 +79,7 @@ BrowserWindow.prototype._init = function () {
       this.webContents.send('ELECTRON_RENDERER_WINDOW_VISIBILITY_CHANGE', isVisible ? 'visible' : 'hidden')
     }
   }
-  
+
   const visibilityEvents = ['show', 'hide', 'minimize', 'maximize', 'restore']
   for (let event of visibilityEvents) {
     this.on(event, visibilityChanged)
@@ -101,6 +101,7 @@ BrowserWindow.getFocusedWindow = () => {
   for (let window of BrowserWindow.getAllWindows()) {
     if (window.isFocused()) return window
   }
+  return null
 }
 
 BrowserWindow.fromWebContents = (webContents) => {
@@ -118,20 +119,20 @@ BrowserWindow.fromDevToolsWebContents = (webContents) => {
 // Helpers.
 
 Object.assign(BrowserWindow.prototype, {
-  loadURL () {
-    return this.webContents.loadURL.apply(this.webContents, arguments)
+  loadURL (...args) {
+    return this.webContents.loadURL.apply(this.webContents, args)
   },
-  getURL () {
+  getURL (...args) {
     return this.webContents.getURL()
   },
-  reload () {
-    return this.webContents.reload.apply(this.webContents, arguments)
+  reload (...args) {
+    return this.webContents.reload.apply(this.webContents, args)
   },
-  send () {
-    return this.webContents.send.apply(this.webContents, arguments)
+  send (...args) {
+    return this.webContents.send.apply(this.webContents, args)
   },
-  openDevTools () {
-    return this.webContents.openDevTools.apply(this.webContents, arguments)
+  openDevTools (...args) {
+    return this.webContents.openDevTools.apply(this.webContents, args)
   },
   closeDevTools () {
     return this.webContents.closeDevTools()
@@ -145,8 +146,8 @@ Object.assign(BrowserWindow.prototype, {
   toggleDevTools () {
     return this.webContents.toggleDevTools()
   },
-  inspectElement () {
-    return this.webContents.inspectElement.apply(this.webContents, arguments)
+  inspectElement (...args) {
+    return this.webContents.inspectElement.apply(this.webContents, args)
   },
   inspectServiceWorker () {
     return this.webContents.inspectServiceWorker()