Revert "stream: add bytesRead property for readable"

This reverts commit bfb2cd0.

The bytesRead property, as implemented, tracks characters
instead of bytes when using an identity encoding.

Refs: #4372
PR-URL: #4746
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>

doc/api/stream.markdown

@@ -250,11 +250,6 @@ readable: null
 end
 ```
 
-
-#### readable.bytesRead
-
-The amount of read bytes. If `objectMode` is `true`, the value is 0 always.
-
 #### readable.isPaused()
 
 * Return: `Boolean`

lib/_stream_readable.js

@@ -83,8 +83,6 @@ function Readable(options) {
 
   this._readableState = new ReadableState(options, this);
 
-  this.bytesRead = 0;
-
   // legacy
   this.readable = true;
 
@@ -137,7 +135,6 @@ function readableAddChunk(stream, state, chunk, encoding, addToFront) {
       var e = new Error('stream.unshift() after end event');
       stream.emit('error', e);
     } else {
-      stream.bytesRead += state.objectMode ? 0 : chunk.length;
       if (state.decoder && !addToFront && !encoding)
         chunk = state.decoder.write(chunk);
 

lib/net.js

@@ -91,6 +91,7 @@ exports._normalizeConnectArgs = normalizeConnectArgs;
 // called when creating new Socket, or when re-using a closed Socket
 function initSocketHandle(self) {
   self.destroyed = false;
+  self.bytesRead = 0;
   self._bytesDispatched = 0;
   self._sockname = null;
 
@@ -514,6 +515,10 @@ function onread(nread, buffer) {
     // will prevent this from being called again until _read() gets
     // called again.
 
+    // if it's not enough data, we'll just call handle.readStart()
+    // again right away.
+    self.bytesRead += nread;
+
     // Optimization: emit the original buffer with end points
     var ret = self.push(buffer);