Merge pull request #4233 from atom/update-node

Ignore errors happened in Node's stdout/stdin stream

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() {
+  describe('vm.createContext', function() {
-      return require('vm').runInNewContext('');
+    it('should not crash', function() {
+      require('vm').runInNewContext('');
     });
   });
 });

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);
 });

vendor/node

@@ -1 +1 @@
-Subproject commit 3b044608ee51ca001dabe944cade6e64f46b0724
+Subproject commit a130651f868f5ad23cb366abacea02f9ed50b769