Merge pull request #5 from atom/master

update as upstream

atom/browser/api/lib/auto-updater/auto-updater-win.js

@@ -3,22 +3,24 @@
 const app = require('electron').app;
 const EventEmitter = require('events').EventEmitter;
 const squirrelUpdate = require('./squirrel-update-win');
+const util = require('util');
 
-class AutoUpdater extends EventEmitter {
-  constructor() {
-    super();
-  }
+function AutoUpdater() {
+  EventEmitter.call(this);
+}
 
-  quitAndInstall() {
+util.inherits(AutoUpdater, EventEmitter);
+
+AutoUpdater.prototype.quitAndInstall = function() {
   squirrelUpdate.processStart();
   return app.quit();
-  }
+};
 
-  setFeedURL(updateURL) {
+AutoUpdater.prototype.setFeedURL = function(updateURL) {
   return this.updateURL = updateURL;
-  }
+};
 
-  checkForUpdates() {
+AutoUpdater.prototype.checkForUpdates = function() {
   if (!this.updateURL) {
     return this.emitError('Update URL is not set');
   }
@@ -50,13 +52,12 @@ class AutoUpdater extends EventEmitter {
       });
     };
   })(this));
-  }
+};
 
-  // Private: Emit both error object and message, this is to keep compatibility
-  // with Old APIs.
-  emitError(message) {
+// Private: Emit both error object and message, this is to keep compatibility
+// with Old APIs.
+AutoUpdater.prototype.emitError = function(message) {
   return this.emit('error', new Error(message), message);
-  }
-}
+};
 
 module.exports = new AutoUpdater;

atom/browser/api/lib/web-contents.js

@@ -76,8 +76,11 @@ let wrapWebContents = function(webContents) {
 
   // WebContents::send(channel, args..)
   webContents.send = function() {
-    var args, channel;
-    channel = arguments[0], args = 2 <= arguments.length ? slice.call(arguments, 1) : [];
+    var args = 2 <= arguments.length ? slice.call(arguments, 1) : [];
+    var channel = arguments[0];
+    if (channel == null) {
+      throw new Error('Missing required channel argument');
+    }
     return this._send(channel, slice.call(args));
   };
 

atom/renderer/lib/web-view/web-view.js

@@ -404,10 +404,13 @@ var registerWebViewElement = function() {
   // Forward proto.foo* method calls to WebViewImpl.foo*.
   createBlockHandler = function(m) {
     return function() {
-      var args, internal, ref1;
-      args = 1 <= arguments.length ? slice.call(arguments, 0) : [];
-      internal = v8Util.getHiddenValue(this, 'internal');
-      return (ref1 = internal.webContents)[m].apply(ref1, args);
+      var args = 1 <= arguments.length ? slice.call(arguments, 0) : [];
+      var internal = v8Util.getHiddenValue(this, 'internal');
+      if (internal.webContents) {
+        return internal.webContents[m].apply(internal.webContents, args);
+      } else {
+        throw new Error(`Cannot call ${m} because the webContents is unavailable. The WebView must be attached to the DOM and the dom-ready event emitted before this method can be called.`);
+      }
     };
   };
   for (i = 0, len = methods.length; i < len; i++) {

package.json

@@ -11,6 +11,8 @@
   },
   "private": true,
   "scripts": {
+    "bootstrap": "python ./script/bootstrap.py",
+    "build": "python ./script/build.py -c D",
     "lint": "python ./script/eslint.py && python ./script/cpplint.py",
     "preinstall": "node -e 'process.exit(0)'",
     "start": "python ./script/start.py",

spec/api-auto-updater-spec.js

@@ -0,0 +1,34 @@
+const assert = require('assert');
+const autoUpdater = require('electron').remote.autoUpdater;
+const ipcRenderer = require('electron').ipcRenderer;
+
+describe('autoUpdater module', function() {
+  describe('checkForUpdates', function() {
+    it('emits an error on Windows when called the feed URL is not set', function (done) {
+      if (process.platform !== 'win32') {
+        return done();
+      }
+
+      ipcRenderer.once('auto-updater-error', function(event, message) {
+        assert.equal(message, 'Update URL is not set');
+        done();
+      });
+      autoUpdater.setFeedURL('');
+      autoUpdater.checkForUpdates();
+    });
+  });
+
+  describe('setFeedURL', function() {
+    it('emits an error on Mac OS X when the application is unsigned', function (done) {
+      if (process.platform !== 'darwin') {
+        return done();
+      }
+
+      ipcRenderer.once('auto-updater-error', function(event, message) {
+        assert.equal(message, 'Could not get code signature for running application');
+        done();
+      });
+      autoUpdater.setFeedURL('');
+    });
+  });
+});

spec/api-browser-window-spec.js

@@ -14,29 +14,31 @@ const BrowserWindow = remote.require('electron').BrowserWindow;
 const isCI = remote.getGlobal('isCi');
 
 describe('browser-window module', function() {
-  var fixtures, w;
-  fixtures = path.resolve(__dirname, 'fixtures');
-  w = null;
+  var fixtures = path.resolve(__dirname, 'fixtures');
+  var w = null;
+
   beforeEach(function() {
     if (w != null) {
       w.destroy();
     }
-    return w = new BrowserWindow({
+    w = new BrowserWindow({
       show: false,
       width: 400,
       height: 400
     });
   });
+
   afterEach(function() {
     if (w != null) {
       w.destroy();
     }
-    return w = null;
+    w = null;
   });
+
   describe('BrowserWindow.close()', function() {
     it('should emit unload handler', function(done) {
       w.webContents.on('did-finish-load', function() {
-        return w.close();
+        w.close();
       });
       w.on('closed', function() {
         var content, test;
@@ -44,20 +46,22 @@ describe('browser-window module', function() {
         content = fs.readFileSync(test);
         fs.unlinkSync(test);
         assert.equal(String(content), 'unload');
-        return done();
+        done();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'unload.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'unload.html'));
     });
-    return it('should emit beforeunload handler', function(done) {
+
+    it('should emit beforeunload handler', function(done) {
       w.on('onbeforeunload', function() {
-        return done();
+        done();
       });
       w.webContents.on('did-finish-load', function() {
-        return w.close();
+        w.close();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'beforeunload-false.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'beforeunload-false.html'));
     });
   });
+
   describe('window.close()', function() {
     it('should emit unload handler', function(done) {
       w.on('closed', function() {
@@ -66,104 +70,113 @@ describe('browser-window module', function() {
         content = fs.readFileSync(test);
         fs.unlinkSync(test);
         assert.equal(String(content), 'close');
-        return done();
+        done();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'close.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'close.html'));
     });
-    return it('should emit beforeunload handler', function(done) {
+
+    it('should emit beforeunload handler', function(done) {
       w.on('onbeforeunload', function() {
-        return done();
+        done();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-false.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-false.html'));
     });
   });
+
   describe('BrowserWindow.destroy()', function() {
-    return it('prevents users to access methods of webContents', function() {
+    it('prevents users to access methods of webContents', function() {
       var webContents;
       webContents = w.webContents;
       w.destroy();
-      return assert.throws((function() {
-        return webContents.getId();
+      assert.throws((function() {
+        webContents.getId();
       }), /Object has been destroyed/);
     });
   });
+
   describe('BrowserWindow.loadURL(url)', function() {
     it('should emit did-start-loading event', function(done) {
       w.webContents.on('did-start-loading', function() {
-        return done();
+        done();
       });
-      return w.loadURL('about:blank');
+      w.loadURL('about:blank');
     });
-    return it('should emit did-fail-load event', function(done) {
+
+    it('should emit did-fail-load event', function(done) {
       w.webContents.on('did-fail-load', function() {
-        return done();
+        done();
       });
-      return w.loadURL('file://a.txt');
+      w.loadURL('file://a.txt');
     });
   });
+
   describe('BrowserWindow.show()', function() {
-    return it('should focus on window', function() {
+    it('should focus on window', function() {
       if (isCI) {
         return;
       }
+
       w.show();
-      return assert(w.isFocused());
+      assert(w.isFocused());
     });
   });
+
   describe('BrowserWindow.showInactive()', function() {
-    return it('should not focus on window', function() {
+    it('should not focus on window', function() {
       w.showInactive();
-      return assert(!w.isFocused());
+      assert(!w.isFocused());
     });
   });
+
   describe('BrowserWindow.focus()', function() {
-    return it('does not make the window become visible', function() {
+    it('does not make the window become visible', function() {
       assert.equal(w.isVisible(), false);
       w.focus();
-      return assert.equal(w.isVisible(), false);
+      assert.equal(w.isVisible(), false);
     });
   });
+
   describe('BrowserWindow.capturePage(rect, callback)', function() {
-    return it('calls the callback with a Buffer', function(done) {
-      return w.capturePage({
+    it('calls the callback with a Buffer', function(done) {
+      w.capturePage({
         x: 0,
         y: 0,
         width: 100,
         height: 100
       }, function(image) {
         assert.equal(image.isEmpty(), true);
-        return done();
+        done();
       });
     });
   });
+
   describe('BrowserWindow.setSize(width, height)', function() {
-    return it('sets the window size', function(done) {
-      var size;
-      size = [300, 400];
+    it('sets the window size', function(done) {
+      var size = [300, 400];
       w.once('resize', function() {
-        var newSize;
-        newSize = w.getSize();
+        var newSize = w.getSize();
         assert.equal(newSize[0], size[0]);
         assert.equal(newSize[1], size[1]);
-        return done();
+        done();
       });
-      return w.setSize(size[0], size[1]);
+      w.setSize(size[0], size[1]);
     });
   });
+
   describe('BrowserWindow.setPosition(x, y)', function() {
-    return it('sets the window position', function(done) {
-      var pos;
-      pos = [10, 10];
+    it('sets the window position', function(done) {
+      var pos = [10, 10];
       w.once('move', function() {
         var newPos;
         newPos = w.getPosition();
         assert.equal(newPos[0], pos[0]);
         assert.equal(newPos[1], pos[1]);
-        return done();
+        done();
       });
-      return w.setPosition(pos[0], pos[1]);
+      w.setPosition(pos[0], pos[1]);
     });
   });
+
   describe('BrowserWindow.setContentSize(width, height)', function() {
     it('sets the content size', function() {
       var after, size;
@@ -171,9 +184,10 @@ describe('browser-window module', function() {
       w.setContentSize(size[0], size[1]);
       after = w.getContentSize();
       assert.equal(after[0], size[0]);
-      return assert.equal(after[1], size[1]);
+      assert.equal(after[1], size[1]);
     });
-    return it('works for framless window', function() {
+
+    it('works for framless window', function() {
       var after, size;
       w.destroy();
       w = new BrowserWindow({
@@ -186,14 +200,16 @@ describe('browser-window module', function() {
       w.setContentSize(size[0], size[1]);
       after = w.getContentSize();
       assert.equal(after[0], size[0]);
-      return assert.equal(after[1], size[1]);
+      assert.equal(after[1], size[1]);
     });
   });
+
   describe('BrowserWindow.fromId(id)', function() {
-    return it('returns the window with id', function() {
-      return assert.equal(w.id, BrowserWindow.fromId(w.id).id);
+    it('returns the window with id', function() {
+      assert.equal(w.id, BrowserWindow.fromId(w.id).id);
     });
   });
+
   describe('"useContentSize" option', function() {
     it('make window created with content size when used', function() {
       var contentSize;
@@ -206,15 +222,16 @@ describe('browser-window module', function() {
       });
       contentSize = w.getContentSize();
       assert.equal(contentSize[0], 400);
-      return assert.equal(contentSize[1], 400);
+      assert.equal(contentSize[1], 400);
     });
+
     it('make window created with window size when not used', function() {
-      var size;
-      size = w.getSize();
+      var size = w.getSize();
       assert.equal(size[0], 400);
-      return assert.equal(size[1], 400);
+      assert.equal(size[1], 400);
     });
-    return it('works for framless window', function() {
+
+    it('works for framless window', function() {
       var contentSize, size;
       w.destroy();
       w = new BrowserWindow({
@@ -229,9 +246,10 @@ describe('browser-window module', function() {
       assert.equal(contentSize[1], 400);
       size = w.getSize();
       assert.equal(size[0], 400);
-      return assert.equal(size[1], 400);
+      assert.equal(size[1], 400);
     });
   });
+
   describe('"title-bar-style" option', function() {
     if (process.platform !== 'darwin') {
       return;
@@ -239,6 +257,7 @@ describe('browser-window module', function() {
     if (parseInt(os.release().split('.')[0]) < 14) {
       return;
     }
+
     it('creates browser window with hidden title bar', function() {
       var contentSize;
       w.destroy();
@@ -249,9 +268,10 @@ describe('browser-window module', function() {
         titleBarStyle: 'hidden'
       });
       contentSize = w.getContentSize();
-      return assert.equal(contentSize[1], 400);
+      assert.equal(contentSize[1], 400);
     });
-    return it('creates browser window with hidden inset title bar', function() {
+
+    it('creates browser window with hidden inset title bar', function() {
       var contentSize;
       w.destroy();
       w = new BrowserWindow({
@@ -261,30 +281,34 @@ describe('browser-window module', function() {
         titleBarStyle: 'hidden-inset'
       });
       contentSize = w.getContentSize();
-      return assert.equal(contentSize[1], 400);
+      assert.equal(contentSize[1], 400);
     });
   });
+
   describe('"enableLargerThanScreen" option', function() {
     if (process.platform === 'linux') {
       return;
     }
+
     beforeEach(function() {
       w.destroy();
-      return w = new BrowserWindow({
+      w = new BrowserWindow({
         show: true,
         width: 400,
         height: 400,
         enableLargerThanScreen: true
       });
     });
+
     it('can move the window out of screen', function() {
       var after;
       w.setPosition(-10, -10);
       after = w.getPosition();
       assert.equal(after[0], -10);
-      return assert.equal(after[1], -10);
+      assert.equal(after[1], -10);
     });
-    return it('can set the window larger than screen', function() {
+
+    it('can set the window larger than screen', function() {
       var after, size;
       size = screen.getPrimaryDisplay().size;
       size.width += 100;
@@ -292,21 +316,22 @@ describe('browser-window module', function() {
       w.setSize(size.width, size.height);
       after = w.getSize();
       assert.equal(after[0], size.width);
-      return assert.equal(after[1], size.height);
+      assert.equal(after[1], size.height);
     });
   });
 
   describe('"web-preferences" option', function() {
     afterEach(function() {
-      return ipcMain.removeAllListeners('answer');
+      ipcMain.removeAllListeners('answer');
     });
+
     describe('"preload" option', function() {
-      return it('loads the script before other scripts in window', function(done) {
+      it('loads the script before other scripts in window', function(done) {
         var preload;
         preload = path.join(fixtures, 'module', 'set-global.js');
         ipcMain.once('answer', function(event, test) {
           assert.equal(test, 'preload');
-          return done();
+          done();
         });
         w.destroy();
         w = new BrowserWindow({
@@ -315,16 +340,17 @@ describe('browser-window module', function() {
             preload: preload
           }
         });
-        return w.loadURL('file://' + path.join(fixtures, 'api', 'preload.html'));
+        w.loadURL('file://' + path.join(fixtures, 'api', 'preload.html'));
       });
     });
-    return describe('"node-integration" option', function() {
-      return it('disables node integration when specified to false', function(done) {
+
+    describe('"node-integration" option', function() {
+      it('disables node integration when specified to false', function(done) {
         var preload;
         preload = path.join(fixtures, 'module', 'send-later.js');
         ipcMain.once('answer', function(event, test) {
           assert.equal(test, 'undefined');
-          return done();
+          done();
         });
         w.destroy();
         w = new BrowserWindow({
@@ -334,107 +360,121 @@ describe('browser-window module', function() {
             nodeIntegration: false
           }
         });
-        return w.loadURL('file://' + path.join(fixtures, 'api', 'blank.html'));
+        w.loadURL('file://' + path.join(fixtures, 'api', 'blank.html'));
       });
     });
   });
+
   describe('beforeunload handler', function() {
     it('returning true would not prevent close', function(done) {
       w.on('closed', function() {
-        return done();
+        done();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-true.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-true.html'));
     });
+
     it('returning non-empty string would not prevent close', function(done) {
       w.on('closed', function() {
-        return done();
+        done();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-string.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-string.html'));
     });
+
     it('returning false would prevent close', function(done) {
       w.on('onbeforeunload', function() {
-        return done();
+        done();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-false.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-false.html'));
     });
-    return it('returning empty string would prevent close', function(done) {
+
+    it('returning empty string would prevent close', function(done) {
       w.on('onbeforeunload', function() {
-        return done();
+        done();
       });
-      return w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-empty-string.html'));
+      w.loadURL('file://' + path.join(fixtures, 'api', 'close-beforeunload-empty-string.html'));
     });
   });
+
   describe('new-window event', function() {
     if (isCI && process.platform === 'darwin') {
       return;
     }
+
     it('emits when window.open is called', function(done) {
       w.webContents.once('new-window', function(e, url, frameName) {
         e.preventDefault();
         assert.equal(url, 'http://host/');
         assert.equal(frameName, 'host');
-        return done();
+        done();
       });
-      return w.loadURL("file://" + fixtures + "/pages/window-open.html");
+      w.loadURL("file://" + fixtures + "/pages/window-open.html");
     });
-    return it('emits when link with target is called', function(done) {
+
+    it('emits when link with target is called', function(done) {
       this.timeout(10000);
       w.webContents.once('new-window', function(e, url, frameName) {
         e.preventDefault();
         assert.equal(url, 'http://host/');
         assert.equal(frameName, 'target');
-        return done();
+        done();
       });
-      return w.loadURL("file://" + fixtures + "/pages/target-name.html");
+      w.loadURL("file://" + fixtures + "/pages/target-name.html");
     });
   });
+
   describe('maximize event', function() {
     if (isCI) {
       return;
     }
-    return it('emits when window is maximized', function(done) {
+
+    it('emits when window is maximized', function(done) {
       this.timeout(10000);
       w.once('maximize', function() {
-        return done();
+        done();
       });
       w.show();
-      return w.maximize();
+      w.maximize();
     });
   });
+
   describe('unmaximize event', function() {
     if (isCI) {
       return;
     }
-    return it('emits when window is unmaximized', function(done) {
+
+    it('emits when window is unmaximized', function(done) {
       this.timeout(10000);
       w.once('unmaximize', function() {
-        return done();
+        done();
       });
       w.show();
       w.maximize();
-      return w.unmaximize();
+      w.unmaximize();
     });
   });
+
   describe('minimize event', function() {
     if (isCI) {
       return;
     }
-    return it('emits when window is minimized', function(done) {
+
+    it('emits when window is minimized', function(done) {
       this.timeout(10000);
       w.once('minimize', function() {
-        return done();
+        done();
       });
       w.show();
-      return w.minimize();
+      w.minimize();
     });
   });
+
   xdescribe('beginFrameSubscription method', function() {
-    return it('subscribes frame updates', function(done) {
+    it('subscribes frame updates', function(done) {
       w.loadURL("file://" + fixtures + "/api/blank.html");
-      return w.webContents.beginFrameSubscription(function(data) {
+      w.webContents.beginFrameSubscription(function(data) {
         assert.notEqual(data.length, 0);
         w.webContents.endFrameSubscription();
-        return done();
+        done();
       });
     });
   });
@@ -472,13 +512,13 @@ describe('browser-window module', function() {
   });
 
   describe('BrowserWindow options argument is optional', function() {
-    return it('should create a window with default size (800x600)', function() {
+    it('should create a window with default size (800x600)', function() {
       var size;
       w.destroy();
       w = new BrowserWindow();
       size = w.getSize();
       assert.equal(size[0], 800);
-      return assert.equal(size[1], 600);
+      assert.equal(size[1], 600);
     });
   });
 
@@ -618,4 +658,16 @@ describe('browser-window module', function() {
       });
     });
   });
+
+  describe('window.webContents.send(channel, args...)', function() {
+    it('throws an error when the channel is missing', function() {
+      assert.throws(function() {
+        w.webContents.send();
+      }, 'Missing required channel argument');
+
+      assert.throws(function() {
+        w.webContents.send(null);
+      }, 'Missing required channel argument');
+    });
+  });
 });

spec/static/main.js

@@ -72,6 +72,11 @@ app.on('ready', function() {
   // Test if using protocol module would crash.
   electron.protocol.registerStringProtocol('test-if-crashes', function() {});
 
+  // Send auto updater errors to window to be verified in specs
+  electron.autoUpdater.on('error', function (error) {
+    window.send('auto-updater-error', error.message)
+  });
+
   window = new BrowserWindow({
     title: 'Electron Tests',
     show: false,

spec/webview-spec.js

@@ -1,166 +1,175 @@
-var assert, http, path, url;
-
-assert = require('assert');
-
-path = require('path');
-
-http = require('http');
-
-url = require('url');
+const assert = require('assert');
+const path = require('path');
+const http = require('http');
+const url = require('url');
 
 describe('<webview> tag', function() {
-  var fixtures, webview;
   this.timeout(10000);
-  fixtures = path.join(__dirname, 'fixtures');
-  webview = null;
+
+  var fixtures = path.join(__dirname, 'fixtures');
+  var webview = null;
+
   beforeEach(function() {
-    return webview = new WebView;
+    webview = new WebView;
   });
+
   afterEach(function() {
     if (document.body.contains(webview)) {
-      return document.body.removeChild(webview);
+      document.body.removeChild(webview);
     }
   });
+
   describe('src attribute', function() {
     it('specifies the page to load', function(done) {
       webview.addEventListener('console-message', function(e) {
         assert.equal(e.message, 'a');
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/a.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('navigates to new page when changed', function(done) {
+
+    it('navigates to new page when changed', function(done) {
       var listener = function() {
         webview.src = "file://" + fixtures + "/pages/b.html";
         webview.addEventListener('console-message', function(e) {
           assert.equal(e.message, 'b');
-          return done();
+          done();
         });
-        return webview.removeEventListener('did-finish-load', listener);
+        webview.removeEventListener('did-finish-load', listener);
       };
       webview.addEventListener('did-finish-load', listener);
       webview.src = "file://" + fixtures + "/pages/a.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('nodeintegration attribute', function() {
     it('inserts no node symbols when not set', function(done) {
       webview.addEventListener('console-message', function(e) {
         assert.equal(e.message, 'undefined undefined undefined undefined');
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/c.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
+
     it('inserts node symbols when set', function(done) {
       webview.addEventListener('console-message', function(e) {
         assert.equal(e.message, 'function object object');
-        return done();
+        done();
       });
       webview.setAttribute('nodeintegration', 'on');
       webview.src = "file://" + fixtures + "/pages/d.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
+
     it('loads node symbols after POST navigation when set', function(done) {
       webview.addEventListener('console-message', function(e) {
         assert.equal(e.message, 'function object object');
-        return done();
+        done();
       });
       webview.setAttribute('nodeintegration', 'on');
       webview.src = "file://" + fixtures + "/pages/post.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
+
     if (process.platform !== 'win32' || process.execPath.toLowerCase().indexOf('\\out\\d\\') === -1) {
-      return it('loads native modules when navigation happens', function(done) {
+      it('loads native modules when navigation happens', function(done) {
         var listener = function() {
           webview.removeEventListener('did-finish-load', listener);
           var listener2 = function(e) {
             assert.equal(e.message, 'function');
-            return done();
+            done();
           };
           webview.addEventListener('console-message', listener2);
-          return webview.reload();
+          webview.reload();
         };
         webview.addEventListener('did-finish-load', listener);
         webview.setAttribute('nodeintegration', 'on');
         webview.src = "file://" + fixtures + "/pages/native-module.html";
-        return document.body.appendChild(webview);
+        document.body.appendChild(webview);
       });
     }
   });
+
   describe('preload attribute', function() {
     it('loads the script before other scripts in window', function(done) {
       var listener;
       listener = function(e) {
         assert.equal(e.message, 'function object object');
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('console-message', listener);
       webview.setAttribute('preload', fixtures + "/module/preload.js");
       webview.src = "file://" + fixtures + "/pages/e.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
+
     it('preload script can still use "process" in required modules when nodeintegration is off', function(done) {
       webview.addEventListener('console-message', function(e) {
         assert.equal(e.message, 'object undefined object');
-        return done();
+        done();
       });
       webview.setAttribute('preload', fixtures + "/module/preload-node-off.js");
       webview.src = "file://" + fixtures + "/api/blank.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('receives ipc message in preload script', function(done) {
+
+    it('receives ipc message in preload script', function(done) {
       var listener, listener2, message;
       message = 'boom!';
       listener = function(e) {
         assert.equal(e.channel, 'pong');
         assert.deepEqual(e.args, [message]);
         webview.removeEventListener('ipc-message', listener);
-        return done();
+        done();
       };
       listener2 = function() {
         webview.send('ping', message);
-        return webview.removeEventListener('did-finish-load', listener2);
+        webview.removeEventListener('did-finish-load', listener2);
       };
       webview.addEventListener('ipc-message', listener);
       webview.addEventListener('did-finish-load', listener2);
       webview.setAttribute('preload', fixtures + "/module/preload-ipc.js");
       webview.src = "file://" + fixtures + "/pages/e.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('httpreferrer attribute', function() {
-    return it('sets the referrer url', function(done) {
+    it('sets the referrer url', function(done) {
       var listener, referrer;
       referrer = 'http://github.com/';
       listener = function(e) {
         assert.equal(e.message, referrer);
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('console-message', listener);
       webview.setAttribute('httpreferrer', referrer);
       webview.src = "file://" + fixtures + "/pages/referrer.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('useragent attribute', function() {
-    return it('sets the user agent', function(done) {
+    it('sets the user agent', function(done) {
       var listener, referrer;
       referrer = 'Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; AS; rv:11.0) like Gecko';
       listener = function(e) {
         assert.equal(e.message, referrer);
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('console-message', listener);
       webview.setAttribute('useragent', referrer);
       webview.src = "file://" + fixtures + "/pages/useragent.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('disablewebsecurity attribute', function() {
     it('does not disable web security when not set', function(done) {
       var encoded, listener, src;
@@ -169,164 +178,177 @@ describe('<webview> tag', function() {
       listener = function(e) {
         assert(/Not allowed to load local resource/.test(e.message));
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('console-message', listener);
       webview.src = "data:text/html;base64," + encoded;
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('disables web security when set', function(done) {
+
+    it('disables web security when set', function(done) {
       var encoded, listener, src;
       src = "<script src='file://" + __dirname + "/static/jquery-2.0.3.min.js'></script> <script>console.log('ok');</script>";
       encoded = btoa(unescape(encodeURIComponent(src)));
       listener = function(e) {
         assert.equal(e.message, 'ok');
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('console-message', listener);
       webview.setAttribute('disablewebsecurity', '');
       webview.src = "data:text/html;base64," + encoded;
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
   describe('partition attribute', function() {
     it('inserts no node symbols when not set', function(done) {
       webview.addEventListener('console-message', function(e) {
         assert.equal(e.message, 'undefined undefined undefined undefined');
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/c.html";
       webview.partition = 'test1';
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
+
     it('inserts node symbols when set', function(done) {
       webview.addEventListener('console-message', function(e) {
         assert.equal(e.message, 'function object object');
-        return done();
+        done();
       });
       webview.setAttribute('nodeintegration', 'on');
       webview.src = "file://" + fixtures + "/pages/d.html";
       webview.partition = 'test2';
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
+
     it('isolates storage for different id', function(done) {
       var listener;
       listener = function(e) {
         assert.equal(e.message, " 0");
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       window.localStorage.setItem('test', 'one');
       webview.addEventListener('console-message', listener);
       webview.src = "file://" + fixtures + "/pages/partition/one.html";
       webview.partition = 'test3';
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('uses current session storage when no id is provided', function(done) {
+
+    it('uses current session storage when no id is provided', function(done) {
       var listener;
       listener = function(e) {
         assert.equal(e.message, "one 1");
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       window.localStorage.setItem('test', 'one');
       webview.addEventListener('console-message', listener);
       webview.src = "file://" + fixtures + "/pages/partition/one.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('allowpopups attribute', function() {
     it('can not open new window when not set', function(done) {
       var listener;
       listener = function(e) {
         assert.equal(e.message, 'null');
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('console-message', listener);
       webview.src = "file://" + fixtures + "/pages/window-open-hide.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('can open new window when set', function(done) {
+
+    it('can open new window when set', function(done) {
       var listener;
       listener = function(e) {
         assert.equal(e.message, 'window');
         webview.removeEventListener('console-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('console-message', listener);
       webview.setAttribute('allowpopups', 'on');
       webview.src = "file://" + fixtures + "/pages/window-open-hide.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('new-window event', function() {
     it('emits when window.open is called', function(done) {
       webview.addEventListener('new-window', function(e) {
         assert.equal(e.url, 'http://host/');
         assert.equal(e.frameName, 'host');
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/window-open.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('emits when link with target is called', function(done) {
+
+    it('emits when link with target is called', function(done) {
       webview.addEventListener('new-window', function(e) {
         assert.equal(e.url, 'http://host/');
         assert.equal(e.frameName, 'target');
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/target-name.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('ipc-message event', function() {
-    return it('emits when guest sends a ipc message to browser', function(done) {
+    it('emits when guest sends a ipc message to browser', function(done) {
       webview.addEventListener('ipc-message', function(e) {
         assert.equal(e.channel, 'channel');
         assert.deepEqual(e.args, ['arg1', 'arg2']);
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/ipc-message.html";
       webview.setAttribute('nodeintegration', 'on');
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('page-title-set event', function() {
-    return it('emits when title is set', function(done) {
+    it('emits when title is set', function(done) {
       webview.addEventListener('page-title-set', function(e) {
         assert.equal(e.title, 'test');
         assert(e.explicitSet);
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/a.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('page-favicon-updated event', function() {
-    return it('emits when favicon urls are received', function(done) {
+    it('emits when favicon urls are received', function(done) {
       webview.addEventListener('page-favicon-updated', function(e) {
         var pageUrl;
         assert.equal(e.favicons.length, 2);
         pageUrl = process.platform === 'win32' ? 'file:///C:/favicon.png' : 'file:///favicon.png';
         assert.equal(e.favicons[0], pageUrl);
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/a.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('will-navigate event', function() {
-    return it('emits when a url that leads to oustide of the page is clicked', function(done) {
+    it('emits when a url that leads to oustide of the page is clicked', function(done) {
       webview.addEventListener('will-navigate', function(e) {
         assert.equal(e.url, "http://host/");
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/webview-will-navigate.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('did-navigate event', function() {
     var p, pageUrl;
     p = path.join(fixtures, 'pages', 'webview-will-navigate.html');
@@ -336,13 +358,14 @@ describe('<webview> tag', function() {
       slashes: true,
       pathname: p
     });
-    return it('emits when a url that leads to outside of the page is clicked', function(done) {
+
+    it('emits when a url that leads to outside of the page is clicked', function(done) {
       webview.addEventListener('did-navigate', function(e) {
         assert.equal(e.url, pageUrl);
-        return done();
+        done();
       });
       webview.src = pageUrl;
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
   describe('did-navigate-in-page event', function() {
@@ -357,20 +380,22 @@ describe('<webview> tag', function() {
       });
       webview.addEventListener('did-navigate-in-page', function(e) {
         assert.equal(e.url, pageUrl + "#test_content");
-        return done();
+        done();
       });
       webview.src = pageUrl;
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
+
     it('emits when window.history.replaceState is called', function(done) {
       webview.addEventListener('did-navigate-in-page', function(e) {
         assert.equal(e.url, "http://host/");
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/webview-did-navigate-in-page-with-history.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('emits when window.location.hash is changed', function(done) {
+
+    it('emits when window.location.hash is changed', function(done) {
       var p, pageUrl;
       p = path.join(fixtures, 'pages', 'webview-did-navigate-in-page-with-hash.html');
       p = p.replace(/\\/g, '/');
@@ -381,94 +406,100 @@ describe('<webview> tag', function() {
       });
       webview.addEventListener('did-navigate-in-page', function(e) {
         assert.equal(e.url, pageUrl + "#test");
-        return done();
+        done();
       });
       webview.src = pageUrl;
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('close event', function() {
-    return it('should fire when interior page calls window.close', function(done) {
+    it('should fire when interior page calls window.close', function(done) {
       webview.addEventListener('close', function() {
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/close.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('devtools-opened event', function() {
-    return it('should fire when webview.openDevTools() is called', function(done) {
+    it('should fire when webview.openDevTools() is called', function(done) {
       var listener;
       listener = function() {
         webview.removeEventListener('devtools-opened', listener);
         webview.closeDevTools();
-        return done();
+        done();
       };
       webview.addEventListener('devtools-opened', listener);
       webview.addEventListener('dom-ready', function() {
-        return webview.openDevTools();
+        webview.openDevTools();
       });
       webview.src = "file://" + fixtures + "/pages/base-page.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('devtools-closed event', function() {
-    return it('should fire when webview.closeDevTools() is called', function(done) {
+    it('should fire when webview.closeDevTools() is called', function(done) {
       var listener, listener2;
       listener2 = function() {
         webview.removeEventListener('devtools-closed', listener2);
-        return done();
+        done();
       };
       listener = function() {
         webview.removeEventListener('devtools-opened', listener);
-        return webview.closeDevTools();
+        webview.closeDevTools();
       };
       webview.addEventListener('devtools-opened', listener);
       webview.addEventListener('devtools-closed', listener2);
       webview.addEventListener('dom-ready', function() {
-        return webview.openDevTools();
+        webview.openDevTools();
       });
       webview.src = "file://" + fixtures + "/pages/base-page.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('devtools-focused event', function() {
-    return it('should fire when webview.openDevTools() is called', function(done) {
+    it('should fire when webview.openDevTools() is called', function(done) {
       var listener;
       listener = function() {
         webview.removeEventListener('devtools-focused', listener);
         webview.closeDevTools();
-        return done();
+        done();
       };
       webview.addEventListener('devtools-focused', listener);
       webview.addEventListener('dom-ready', function() {
-        return webview.openDevTools();
+        webview.openDevTools();
       });
       webview.src = "file://" + fixtures + "/pages/base-page.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('<webview>.reload()', function() {
-    return it('should emit beforeunload handler', function(done) {
+    it('should emit beforeunload handler', function(done) {
       var listener, listener2;
       listener = function(e) {
         assert.equal(e.channel, 'onbeforeunload');
         webview.removeEventListener('ipc-message', listener);
-        return done();
+        done();
       };
       listener2 = function() {
         webview.reload();
-        return webview.removeEventListener('did-finish-load', listener2);
+        webview.removeEventListener('did-finish-load', listener2);
       };
       webview.addEventListener('ipc-message', listener);
       webview.addEventListener('did-finish-load', listener2);
       webview.setAttribute('nodeintegration', 'on');
       webview.src = "file://" + fixtures + "/pages/beforeunload-false.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('<webview>.clearHistory()', function() {
-    return it('should clear the navigation history', function(done) {
+    it('should clear the navigation history', function(done) {
       var listener;
       listener = function(e) {
         assert.equal(e.channel, 'history');
@@ -477,18 +508,19 @@ describe('<webview> tag', function() {
         webview.clearHistory();
         assert(!webview.canGoBack());
         webview.removeEventListener('ipc-message', listener);
-        return done();
+        done();
       };
       webview.addEventListener('ipc-message', listener);
       webview.setAttribute('nodeintegration', 'on');
       webview.src = "file://" + fixtures + "/pages/history.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('basic auth', function() {
     var auth;
     auth = require('basic-auth');
-    return it('should authenticate with correct credentials', function(done) {
+    it('should authenticate with correct credentials', function(done) {
       var message, server;
       message = 'Authenticated';
       server = http.createServer(function(req, res) {
@@ -499,67 +531,77 @@ describe('<webview> tag', function() {
         } else {
           res.end('failed');
         }
-        return server.close();
+        server.close();
       });
-      return server.listen(0, '127.0.0.1', function() {
+      server.listen(0, '127.0.0.1', function() {
         var port;
         port = server.address().port;
         webview.addEventListener('ipc-message', function(e) {
           assert.equal(e.channel, message);
-          return done();
+          done();
         });
         webview.src = "file://" + fixtures + "/pages/basic-auth.html?port=" + port;
         webview.setAttribute('nodeintegration', 'on');
-        return document.body.appendChild(webview);
+        document.body.appendChild(webview);
       });
     });
   });
+
   describe('dom-ready event', function() {
-    return it('emits when document is loaded', function(done) {
+    it('emits when document is loaded', function(done) {
       var server;
       server = http.createServer(function() {});
-      return server.listen(0, '127.0.0.1', function() {
+      server.listen(0, '127.0.0.1', function() {
         var port;
         port = server.address().port;
         webview.addEventListener('dom-ready', function() {
-          return done();
+          done();
         });
         webview.src = "file://" + fixtures + "/pages/dom-ready.html?port=" + port;
-        return document.body.appendChild(webview);
+        document.body.appendChild(webview);
       });
     });
+
+    it('throws a custom error when an API method is called before the event is emitted', function() {
+      assert.throws(function () {
+        webview.stop();
+      }, 'Cannot call stop because the webContents is unavailable. The WebView must be attached to the DOM and the dom-ready event emitted before this method can be called.');
     });
+  });
+
   describe('executeJavaScript', function() {
     if (process.env.TRAVIS !== 'true') {
       return;
     }
-    return it('should support user gesture', function(done) {
+
+    it('should support user gesture', function(done) {
       var listener, listener2;
       listener = function() {
         webview.removeEventListener('enter-html-full-screen', listener);
-        return done();
+        done();
       };
       listener2 = function() {
         var jsScript;
         jsScript = 'document.getElementsByTagName("video")[0].webkitRequestFullScreen()';
         webview.executeJavaScript(jsScript, true);
-        return webview.removeEventListener('did-finish-load', listener2);
+        webview.removeEventListener('did-finish-load', listener2);
       };
       webview.addEventListener('enter-html-full-screen', listener);
       webview.addEventListener('did-finish-load', listener2);
       webview.src = "file://" + fixtures + "/pages/fullscreen.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('sendInputEvent', function() {
     it('can send keyboard event', function(done) {
       webview.addEventListener('ipc-message', function(e) {
         assert.equal(e.channel, 'keyup');
         assert.deepEqual(e.args, [67, true, false]);
-        return done();
+        done();
       });
       webview.addEventListener('dom-ready', function() {
-        return webview.sendInputEvent({
+        webview.sendInputEvent({
           type: 'keyup',
           keyCode: 'c',
           modifiers: ['shift']
@@ -567,16 +609,17 @@ describe('<webview> tag', function() {
       });
       webview.src = "file://" + fixtures + "/pages/onkeyup.html";
       webview.setAttribute('nodeintegration', 'on');
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
-    return it('can send mouse event', function(done) {
+
+    it('can send mouse event', function(done) {
       webview.addEventListener('ipc-message', function(e) {
         assert.equal(e.channel, 'mouseup');
         assert.deepEqual(e.args, [10, 20, false, true]);
-        return done();
+        done();
       });
       webview.addEventListener('dom-ready', function() {
-        return webview.sendInputEvent({
+        webview.sendInputEvent({
           type: 'mouseup',
           modifiers: ['ctrl'],
           x: 10,
@@ -585,26 +628,28 @@ describe('<webview> tag', function() {
       });
       webview.src = "file://" + fixtures + "/pages/onmouseup.html";
       webview.setAttribute('nodeintegration', 'on');
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('media-started-playing media-paused events', function() {
-    return it('emits when audio starts and stops playing', function(done) {
+    it('emits when audio starts and stops playing', function(done) {
       var audioPlayed;
       audioPlayed = false;
       webview.addEventListener('media-started-playing', function() {
-        return audioPlayed = true;
+        audioPlayed = true;
       });
       webview.addEventListener('media-paused', function() {
         assert(audioPlayed);
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/audio.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('found-in-page event', function() {
-    return it('emits when a request is made', function(done) {
+    it('emits when a request is made', function(done) {
       var listener, listener2, requestId;
       requestId = null;
       listener = function(e) {
@@ -612,27 +657,29 @@ describe('<webview> tag', function() {
         if (e.result.finalUpdate) {
           assert.equal(e.result.matches, 3);
           webview.stopFindInPage("clearSelection");
-          return done();
+          done();
         }
       };
       listener2 = function() {
-        return requestId = webview.findInPage("virtual");
+        requestId = webview.findInPage("virtual");
       };
       webview.addEventListener('found-in-page', listener);
       webview.addEventListener('did-finish-load', listener2);
       webview.src = "file://" + fixtures + "/pages/content.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   xdescribe('did-change-theme-color event', function() {
-    return it('emits when theme color changes', function(done) {
+    it('emits when theme color changes', function(done) {
       webview.addEventListener('did-change-theme-color', function() {
-        return done();
+        done();
       });
       webview.src = "file://" + fixtures + "/pages/theme-color.html";
-      return document.body.appendChild(webview);
+      document.body.appendChild(webview);
     });
   });
+
   describe('permission-request event', function() {
     function setUpRequestHandler(webview, requested_permission) {
       const session = require('electron').remote.session;
@@ -645,7 +692,7 @@ describe('<webview> tag', function() {
       session.fromPartition(webview.partition).setPermissionRequestHandler(listener);
     }
 
-    it ('emits when using navigator.getUserMedia api', function(done) {
+    it('emits when using navigator.getUserMedia api', function(done) {
       webview.addEventListener('ipc-message', function(e) {
         assert(e.channel, 'message');
         assert(e.args, ['PermissionDeniedError']);
@@ -658,7 +705,7 @@ describe('<webview> tag', function() {
       document.body.appendChild(webview);
     });
 
-    it ('emits when using navigator.geolocation api', function(done) {
+    it('emits when using navigator.geolocation api', function(done) {
       webview.addEventListener('ipc-message', function(e) {
         assert(e.channel, 'message');
         assert(e.args, ['ERROR(1): User denied Geolocation']);
@@ -671,7 +718,7 @@ describe('<webview> tag', function() {
       document.body.appendChild(webview);
     });
 
-    it ('emits when using navigator.requestMIDIAccess api', function(done) {
+    it('emits when using navigator.requestMIDIAccess api', function(done) {
       webview.addEventListener('ipc-message', function(e) {
         assert(e.channel, 'message');
         assert(e.args, ['SecurityError']);