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

PR-URL: #23307
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>

Author: addaleax

lib/internal/bootstrap/node.js

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

lib/zlib.js

@@ -36,7 +36,7 @@ const {
     isArrayBufferView
   }
 } = require('util');
-const binding = process.binding('zlib');
+const binding = internalBinding('zlib');
 const assert = require('assert').ok;
 const {
   Buffer,

src/node_zlib.cc

@@ -850,4 +850,4 @@ void DefineZlibConstants(Local<Object> target) {
 
 }  // namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(zlib, node::Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(zlib, node::Initialize)

test/async-hooks/test-zlib.zlib-binding.deflate.js

@@ -1,3 +1,4 @@
+// Flags: --expose-internals
 'use strict';
 
 const common = require('../common');
@@ -8,7 +9,8 @@ const { checkInvocations } = require('./hook-checks');
 const hooks = initHooks();
 
 hooks.enable();
-const Zlib = process.binding('zlib').Zlib;
+const { internalBinding } = require('internal/test/binding');
+const { Zlib } = internalBinding('zlib');
 const constants = process.binding('constants').zlib;
 
 const handle = new Zlib(constants.DEFLATE);