stream: resolve perf regression introduced by V8 7.3

This commit contains two fixes:
1. use instanceof instead of Object.getPrototypeOf, as checking an
   object prototype with Object.getPrototypeOf is slower
   than an instanceof check.
2. avoid parseInt(undefined, 10) to get NaN as it regressed.

PR-URL: #28842
Fixes: #28586
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Anto Aravinth <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Rich Trott <[email protected]>

Author: mcollina

lib/_stream_readable.js

@@ -250,7 +250,8 @@ function readableAddChunk(stream, chunk, encoding, addToFront) {
     } else if (state.objectMode || chunk && chunk.length > 0) {
       if (typeof chunk !== 'string' &&
           !state.objectMode &&
-          Object.getPrototypeOf(chunk) !== Buffer.prototype) {
+          // Do not use Object.getPrototypeOf as it is slower since V8 7.3.
+          !(chunk instanceof Buffer)) {
         chunk = Stream._uint8ArrayToBuffer(chunk);
       }
 
@@ -393,7 +394,13 @@ function howMuchToRead(n, state) {
 // You can override either this method, or the async _read(n) below.
 Readable.prototype.read = function(n) {
   debug('read', n);
-  n = parseInt(n, 10);
+  // Same as parseInt(undefined, 10), however V8 7.3 performance regressed
+  // in this scenario, so we are doing it manually.
+  if (n === undefined) {
+    n = NaN;
+  } else if (!Number.isInteger(n)) {
+    n = parseInt(n, 10);
+  }
   const state = this._readableState;
   const nOrig = n;
 

lib/_stream_writable.js

@@ -277,7 +277,8 @@ Writable.prototype.write = function(chunk, encoding, cb) {
   var ret = false;
   const isBuf = !state.objectMode && Stream._isUint8Array(chunk);
 
-  if (isBuf && Object.getPrototypeOf(chunk) !== Buffer.prototype) {
+  // Do not use Object.getPrototypeOf as it is slower since V8 7.3.
+  if (isBuf && !(chunk instanceof Buffer)) {
     chunk = Stream._uint8ArrayToBuffer(chunk);
   }