Add child process close event handler

Author: marcbachmann

lib/pdf.js

@@ -63,7 +63,7 @@ PDF.prototype.toStream = function PdfToStream (callback) {
     }
 
     stream.on('end', function () {
-      fs.unlink(res.filename, function (err) {
+      fs.unlink(res.filename, function unlinkPdfFile (err) {
         if (err) console.log('html-pdf:', err)
       })
     })
@@ -134,15 +134,15 @@ PDF.prototype.exec = function PdfExec (callback) {
   child.on('error', function onError (err) { respond(null, err) })
 
   // An exit event is most likely an error because we didn't get any data at this point
+  child.on('close', respond)
   child.on('exit', respond)
 
   var config = JSON.stringify({html: this.html, options: this.options})
-  return child.stdin.write(config + '\n', 'utf8')
+  child.stdin.write(config + '\n', 'utf8')
+  child.stdin.end()
 }
 
 function kill (child, onData, onError) {
-  child.stdout.removeListener('data', onData)
-  child.stderr.removeListener('data', onError)
   child.stdin.end()
   child.kill()
 }