lib: use MIMEType.parse instead of constructor

Signed-off-by: James M Snell <jasnell@gmail.com>

Author: jasnell

doc/api/util.md

@@ -1898,6 +1898,10 @@ console.log(JSON.stringify(myMIMES));
 
 ### `MIMEType.parse(string)`
 
+<!--
+added: REPLACEME
+-->
+
 * `string` {string} The input MIME to parse
 * Returns: {MIMEType|null}
 

lib/internal/data_url.js

@@ -117,13 +117,8 @@ function dataURLProcessor(dataURL) {
   // mimeType.
   // 14. If mimeTypeRecord is failure, then set
   // mimeTypeRecord to text/plain;charset=US-ASCII.
-  let mimeTypeRecord;
-
-  try {
-    mimeTypeRecord = new MIMEType(mimeType);
-  } catch {
-    mimeTypeRecord = new MIMEType('text/plain;charset=US-ASCII');
-  }
+  const mimeTypeRecord = MIMEType.parse(mimeType) ||
+      new MIMEType('text/plain;charset=US-ASCII');
 
   // 15. Return a new data: URL struct whose MIME
   // type is mimeTypeRecord and body is body.

lib/internal/inspector/network.js

@@ -52,16 +52,9 @@ function getNextRequestId() {
 };
 
 function sniffMimeType(contentType) {
-  let mimeType;
-  let charset;
-  try {
-    const mimeTypeObj = new MIMEType(contentType);
-    mimeType = StringPrototypeToLowerCase(mimeTypeObj.essence || '');
-    charset = StringPrototypeToLowerCase(mimeTypeObj.params.get('charset') || '');
-  } catch {
-    mimeType = '';
-    charset = '';
-  }
+  const mimeTypeObj = MIMEType.parse(contentType);
+  const mimeType = StringPrototypeToLowerCase(mimeTypeObj?.essence || '');
+  const charset = StringPrototypeToLowerCase(mimeTypeObj?.params.get('charset') || '');
 
   return {
     __proto__: null,

lib/internal/mime.js

@@ -343,7 +343,8 @@ class MIMEType {
   #parameters;
   constructor(string, noThrowSymbol = null) {
     string = `${string}`;
-    if (noThrowSymbol != null && typeof noThrowSymbol != 'symbol') {
+    // noThrowSymbol can be null or kNoThrow, but not any other value
+    if (noThrowSymbol != null && noThrowSymbol !== kNoThrow) {
       throw new ERR_ILLEGAL_CONSTRUCTOR();
     }
     const data = parseTypeAndSubtype(string, noThrowSymbol);