nodejs
diff --git a/‎doc/api/child_process.markdown‎
Lines changed: 3 additions & 2 deletions b/‎doc/api/child_process.markdown‎
Lines changed: 3 additions & 2 deletions
diff --git a/‎lib/internal/child_process.js‎
Lines changed: 3 additions & 3 deletions b/‎lib/internal/child_process.js‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎test/parallel/test-child-process-send-returns-boolean.js‎
Lines changed: 9 additions & 0 deletions b/‎test/parallel/test-child-process-send-returns-boolean.js‎
Lines changed: 9 additions & 0 deletions
@@ -266,9 +266,10 @@ argument: `null` on success, or an `Error` object on failure.
 `child.send()` emits an `'error'` event if no callback was given and the message
 cannot be sent, for example because the child process has already exited.
 
-Returns `true` under normal circumstances or `false` when the backlog of
+`child.send()` returns `false` if the channel has closed or when the backlog of
 unsent messages exceeds a threshold that makes it unwise to send more.
-Use the callback mechanism to implement flow control.
+Otherwise, it returns `true`. Use the callback mechanism to implement flow
+control.
 
 #### Example: sending server object
 
 
@@ -504,15 +504,15 @@ function setupChannel(target, channel) {
       handle = undefined;
     }
     if (this.connected) {
-      this._send(message, handle, false, callback);
-      return;
+      return this._send(message, handle, false, callback);
     }
     const ex = new Error('channel closed');
     if (typeof callback === 'function') {
       process.nextTick(callback, ex);
     } else {
       this.emit('error', ex);  // FIXME(bnoordhuis) Defer to next tick.
     }
+    return false;
   };
 
   target._send = function(message, handle, swallowErrors, callback) {
@@ -577,7 +577,7 @@ function setupChannel(target, channel) {
         handle: null,
         message: message,
       });
-      return;
+      return this._handleQueue.length === 1;
     }
 
     var req = new WriteWrap();
 
@@ -0,0 +1,9 @@
+'use strict';
+const common = require('../common');
+const assert = require('assert');
+const fork = require('child_process').fork;
+
+const n = fork(common.fixturesDir + '/empty.js');
+
+const rv = n.send({ hello: 'world' });
+assert.strictEqual(rv, true);
Original file line number	Diff line number	Diff line change
@@ -504,15 +504,15 @@ function setupChannel(target, channel) {
504	504	handle = undefined;
505	505	}
506	506	if (this.connected) {
507		- this._send(message, handle, false, callback);
508		- return;
	507	+ return this._send(message, handle, false, callback);
509	508	}
510	509	const ex = new Error('channel closed');
511	510	if (typeof callback === 'function') {
512	511	process.nextTick(callback, ex);
513	512	} else {
514	513	this.emit('error', ex); // FIXME(bnoordhuis) Defer to next tick.
515	514	}
	515	+ return false;
516	516	};
517	517
518	518	target._send = function(message, handle, swallowErrors, callback) {
@@ -577,7 +577,7 @@ function setupChannel(target, channel) {
577	577	handle: null,
578	578	message: message,
579	579	});
580		- return;
	580	+ return this._handleQueue.length === 1;
581	581	}
582	582
583	583	var req = new WriteWrap();