url: use `hasIntl` instead of `try-catch`

Like the other internal modules, we should use
`process.binding('config').hasIntl` instead of `try-catch`
to make sure `icu` is bonded or not.

PR-URL: https://github.com/nodejs/node/pull/11571
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Jackson Tian <shyvo1987@gmail.com>

1 files changed, 2 insertions, 10 deletions

diff --git a/lib/url.js b/lib/url.js
index 2db780c7eb..1623de56ca 100644
--- a/lib/url.js
+++ b/lib/url.js
@@ -1,15 +1,7 @@
 'use strict';
 
-function importPunycode() {
-  try {
-    return process.binding('icu');
-  } catch (e) {
-    return require('punycode');
-  }
-}
-
-const { toASCII } = importPunycode();
-
+const { toASCII } = process.binding('config').hasIntl ?
+  process.binding('icu') : require('punycode');
 const { StorageObject, hexTable } = require('internal/querystring');
 const internalUrl = require('internal/url');
 exports.parse = urlParse;