Bug 1819350 - [devtools] Simplify prettyFast sanitize function. r=devtools-reviewers,jdescottes

Differential Revision: https://phabricator.services.mozilla.com/D171242

devtools/client/debugger/bin/module-manifest.json

@@ -2507,7 +2507,7 @@
         "byName": {},
         "byBlocks": {},
         "usedIds": {
-          "0": 0
+          "1": 1
         }
       }
     }
@@ -2528,7 +2528,7 @@
         "byName": {},
         "byBlocks": {},
         "usedIds": {
-          "0": 0
+          "1": 1
         }
       }
     }

devtools/client/debugger/dist/pretty-print-worker.js

@@ -6212,44 +6212,45 @@ function prependWhiteSpace(token, lastToken, addedNewline, addedSpace, write, op
     spaceAdded = true;
   }
 }
+
+const escapeCharacters = {
+  // Backslash
+  "\\": "\\\\",
+  // Newlines
+  "\n": "\\n",
+  // Carriage return
+  "\r": "\\r",
+  // Tab
+  "\t": "\\t",
+  // Vertical tab
+  "\v": "\\v",
+  // Form feed
+  "\f": "\\f",
+  // Null character
+  "\0": "\\x00",
+  // Line separator
+  "\u2028": "\\u2028",
+  // Paragraph separator
+  "\u2029": "\\u2029",
+  // Single quotes
+  "'": "\\'"
+}; // eslint-disable-next-line prefer-template
+
+const regExpString = "(" + Object.values(escapeCharacters).join("|") + ")";
+const escapeCharactersRegExp = new RegExp(regExpString, "g");
+
+function sanitizerReplaceFunc(_, c) {
+  return escapeCharacters[c];
+}
 /**
  * Make sure that we output the escaped character combination inside string
  * literals instead of various problematic characters.
  */
 
 
-const sanitize = function () {
+function sanitize(str) {
-  const escapeCharacters = {
+  return str.replace(escapeCharactersRegExp, sanitizerReplaceFunc);
-    // Backslash
+}
-    "\\": "\\\\",
-    // Newlines
-    "\n": "\\n",
-    // Carriage return
-    "\r": "\\r",
-    // Tab
-    "\t": "\\t",
-    // Vertical tab
-    "\v": "\\v",
-    // Form feed
-    "\f": "\\f",
-    // Null character
-    "\0": "\\x00",
-    // Line separator
-    "\u2028": "\\u2028",
-    // Paragraph separator
-    "\u2029": "\\u2029",
-    // Single quotes
-    "'": "\\'"
-  }; // eslint-disable-next-line prefer-template
-
-  const regExpString = "(" + Object.values(escapeCharacters).join("|") + ")";
-  const escapeCharactersRegExp = new RegExp(regExpString, "g");
-  return function (str) {
-    return str.replace(escapeCharactersRegExp, function (_, c) {
-      return escapeCharacters[c];
-    });
-  };
-}();
 /**
  * Add the given token to the pretty printed results.
  *

devtools/client/debugger/src/workers/pretty-print/pretty-fast.js

@@ -537,44 +537,45 @@ function prependWhiteSpace(
   }
 }
 
+const escapeCharacters = {
+  // Backslash
+  "\\": "\\\\",
+  // Newlines
+  "\n": "\\n",
+  // Carriage return
+  "\r": "\\r",
+  // Tab
+  "\t": "\\t",
+  // Vertical tab
+  "\v": "\\v",
+  // Form feed
+  "\f": "\\f",
+  // Null character
+  "\0": "\\x00",
+  // Line separator
+  "\u2028": "\\u2028",
+  // Paragraph separator
+  "\u2029": "\\u2029",
+  // Single quotes
+  "'": "\\'",
+};
+
+// eslint-disable-next-line prefer-template
+const regExpString = "(" + Object.values(escapeCharacters).join("|") + ")";
+const escapeCharactersRegExp = new RegExp(regExpString, "g");
+
+function sanitizerReplaceFunc(_, c) {
+  return escapeCharacters[c];
+}
+
 /**
  * Make sure that we output the escaped character combination inside string
  * literals instead of various problematic characters.
  */
-const sanitize = (function() {
+function sanitize(str) {
-  const escapeCharacters = {
+  return str.replace(escapeCharactersRegExp, sanitizerReplaceFunc);
-    // Backslash
+}
-    "\\": "\\\\",
-    // Newlines
-    "\n": "\\n",
-    // Carriage return
-    "\r": "\\r",
-    // Tab
-    "\t": "\\t",
-    // Vertical tab
-    "\v": "\\v",
-    // Form feed
-    "\f": "\\f",
-    // Null character
-    "\0": "\\x00",
-    // Line separator
-    "\u2028": "\\u2028",
-    // Paragraph separator
-    "\u2029": "\\u2029",
-    // Single quotes
-    "'": "\\'",
-  };
 
-  // eslint-disable-next-line prefer-template
-  const regExpString = "(" + Object.values(escapeCharacters).join("|") + ")";
-  const escapeCharactersRegExp = new RegExp(regExpString, "g");
-
-  return function(str) {
-    return str.replace(escapeCharactersRegExp, function(_, c) {
-      return escapeCharacters[c];
-    });
-  };
-})();
 /**
  * Add the given token to the pretty printed results.
  *