icu: make process.binding('icu') internal

lib/buffer.js

@@ -37,14 +37,7 @@ const {
   kMaxLength,
   kStringMaxLength
 } = process.binding('buffer');
-// We cannot use internalBinding unconditionally here because of the way
-// that test/parallel/test-buffer-bindingobj-no-zerofill.js is written.
-let isAnyArrayBuffer;
-try {
-  isAnyArrayBuffer = internalBinding('types').isAnyArrayBuffer;
-} catch (e) {
-  isAnyArrayBuffer = require('util').types.isAnyArrayBuffer;
-}
+const { isAnyArrayBuffer } = internalBinding('types');
 const {
   customInspectSymbol,
   isInsideNodeModules,
@@ -1078,7 +1071,7 @@ if (process.binding('config').hasIntl) {
   const {
     icuErrName,
     transcode: _transcode
-  } = process.binding('icu');
+  } = internalBinding('icu');
 
   // Transcodes the Buffer from one encoding to another, returning a new
   // Buffer instance.

lib/internal/bootstrap/node.js

@@ -394,6 +394,7 @@
       new SafeSet([
         'cares_wrap',
         'fs_event_wrap',
+        'icu',
         'udp_wrap',
         'uv',
         'pipe_wrap',
@@ -654,7 +655,7 @@
 
   function setupProcessICUVersions() {
     const icu = process.binding('config').hasIntl ?
-      process.binding('icu') : undefined;
+      internalBinding('icu') : undefined;
     if (!icu) return;  // no Intl/ICU: nothing to add here.
     // With no argument, getVersion() returns a comma separated list
     // of possible types.

lib/internal/encoding.js

@@ -356,7 +356,7 @@ function makeTextDecoderICU() {
     decode: _decode,
     getConverter,
     hasConverter
-  } = process.binding('icu');
+  } = internalBinding('icu');
 
   class TextDecoder {
     constructor(encoding = 'utf-8', options = {}) {

lib/internal/readline.js

@@ -31,7 +31,7 @@ CSI.kClearLine = CSI`2K`;
 CSI.kClearScreenDown = CSI`0J`;
 
 if (process.binding('config').hasIntl) {
-  const icu = process.binding('icu');
+  const icu = internalBinding('icu');
   getStringWidth = function getStringWidth(str, options) {
     options = options || {};
     if (!Number.isInteger(str))

lib/url.js

@@ -22,7 +22,7 @@
 'use strict';
 
 const { toASCII } = process.binding('config').hasIntl ?
-  process.binding('icu') : require('punycode');
+  internalBinding('icu') : require('punycode');
 
 const { hexTable } = require('internal/querystring');
 

src/node_i18n.cc

@@ -882,6 +882,6 @@ void Initialize(Local<Object> target,
 }  // namespace i18n
 }  // namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(icu, node::i18n::Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(icu, node::i18n::Initialize)
 
 #endif  // NODE_HAVE_I18N_SUPPORT

test/parallel/test-icu-punycode.js

@@ -1,10 +1,12 @@
 'use strict';
+// Flags: --expose-internals
 const common = require('../common');
 
 if (!common.hasIntl)
   common.skip('missing Intl');
 
-const icu = process.binding('icu');
+const { internalBinding } = require('internal/test/binding');
+const icu = internalBinding('icu');
 const assert = require('assert');
 
 const tests = require('../fixtures/url-idna.js');