Merge pull request electron#4233 from atom/update-node

Ignore errors happened in Node's stdout/stdin stream

Author: zcbenz

Diff for: spec/node-spec.js

@@ -190,17 +190,24 @@ describe('node feature', function() {
       return assert.equal(Buffer.byteLength(p.innerText), 13);
     });
   });
+
   describe('process.stdout', function() {
     it('should not throw exception', function() {
-      return process.stdout;
+      process.stdout;
     });
-    return xit('should have isTTY defined', function() {
-      return assert.equal(typeof process.stdout.isTTY, 'boolean');
+
+    it('should not throw exception when calling write()', function() {
+      process.stdout.write('test');
+    });
+
+    xit('should have isTTY defined', function() {
+      assert.equal(typeof process.stdout.isTTY, 'boolean');
     });
   });
-  return describe('vm.createContext', function() {
-    return it('should not crash', function() {
-      return require('vm').runInNewContext('');
+
+  describe('vm.createContext', function() {
+    it('should not crash', function() {
+      require('vm').runInNewContext('');
     });
   });
 });

Diff for: spec/static/main.js

@@ -28,6 +28,9 @@ app.commandLine.appendSwitch('disable-renderer-backgrounding');
 // sure we can reproduce it in renderer process.
 process.stdout;
 
+// Access console to reproduce #3482.
+console;
+
 ipcMain.on('message', function(event, arg) {
   event.sender.send('message', arg);
 });