buffer: move process.binding('buffer') to internalBinding

PR-URL: #22370
Refs: #22160
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Denys Otrishko <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>

Author: starkwang

lib/buffer.js

@@ -36,7 +36,7 @@ const {
   swap64: _swap64,
   kMaxLength,
   kStringMaxLength
-} = process.binding('buffer');
+} = internalBinding('buffer');
 const { isAnyArrayBuffer } = internalBinding('types');
 const {
   customInspectSymbol,

lib/internal/bootstrap/node.js

@@ -413,7 +413,8 @@
         'url',
         'spawn_sync',
         'js_stream',
-        'zlib']);
+        'zlib',
+        'buffer']);
     process.binding = function binding(name) {
       return internalBindingWhitelist.has(name) ?
         internalBinding(name) :

lib/internal/buffer.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const binding = process.binding('buffer');
+const binding = internalBinding('buffer');
 const {
   ERR_BUFFER_OUT_OF_BOUNDS,
   ERR_INVALID_ARG_TYPE,

lib/internal/encoding.js

@@ -29,7 +29,7 @@ const {
 
 const {
   encodeUtf8String
-} = process.binding('buffer');
+} = internalBinding('buffer');
 
 var Buffer;
 function lazyBuffer() {

lib/internal/errors.js

@@ -20,7 +20,7 @@ const {
   UV_EAI_NODATA,
   UV_EAI_NONAME
 } = internalBinding('uv');
-const { kMaxLength } = process.binding('buffer');
+const { kMaxLength } = internalBinding('buffer');
 const { defineProperty } = Object;
 
 // Lazily loaded

lib/internal/util/comparisons.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const { compare } = process.binding('buffer');
+const { compare } = internalBinding('buffer');
 const { isArrayBufferView } = require('internal/util/types');
 const {
   isAnyArrayBuffer,

lib/v8.js

@@ -20,7 +20,7 @@ const {
   Serializer: _Serializer,
   Deserializer: _Deserializer
 } = internalBinding('serdes');
-const { copy } = process.binding('buffer');
+const { copy } = internalBinding('buffer');
 const { objectToString } = require('internal/util');
 const { FastBuffer } = require('internal/buffer');
 

src/node_buffer.cc

@@ -1104,4 +1104,4 @@ void Initialize(Local<Object> target,
 }  // namespace Buffer
 }  // namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(buffer, node::Buffer::Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(buffer, node::Buffer::Initialize)

test/addons/stringbytes-external-exceed-max/test-stringbytes-external-at-max.js

@@ -1,6 +1,8 @@
 'use strict';
+// Flags: --expose-internals
 
 const common = require('../../common');
+const { internalBinding } = require('internal/test/binding');
 const skipMessage = 'intensive toString tests due to memory confinements';
 if (!common.enoughTestMem)
   common.skip(skipMessage);
@@ -10,7 +12,7 @@ const assert = require('assert');
 
 // v8 fails silently if string length > v8::String::kMaxLength
 // v8::String::kMaxLength defined in v8.h
-const kStringMaxLength = process.binding('buffer').kStringMaxLength;
+const kStringMaxLength = internalBinding('buffer').kStringMaxLength;
 
 let buf;
 try {

test/addons/stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-ascii.js

@@ -1,6 +1,8 @@
 'use strict';
+// Flags: --expose-internals
 
 const common = require('../../common');
+const { internalBinding } = require('internal/test/binding');
 const skipMessage = 'intensive toString tests due to memory confinements';
 if (!common.enoughTestMem)
   common.skip(skipMessage);
@@ -9,7 +11,7 @@ const binding = require(`./build/${common.buildType}/binding`);
 
 // v8 fails silently if string length > v8::String::kMaxLength
 // v8::String::kMaxLength defined in v8.h
-const kStringMaxLength = process.binding('buffer').kStringMaxLength;
+const kStringMaxLength = internalBinding('buffer').kStringMaxLength;
 
 let buf;
 try {