fixup! perf_hooks: reduce overhead of new user timings

Author: H4ad

lib/internal/perf/usertiming.js

@@ -66,17 +66,15 @@ function getMark(name) {
   return ts;
 }
 
-const kEmptyArg = Symbol('kEmptyArg');
-
 class PerformanceMark extends PerformanceEntry {
-  constructor(name = kEmptyArg, options = undefined) {
-    if (name === kEmptyArg) {
+  constructor(name, options = undefined) {
+    if (arguments.length === 0) {
       throw new ERR_MISSING_ARGS('name');
     }
     name = `${name}`;
     if (nodeTimingReadOnlyAttributes.has(name))
       throw new ERR_INVALID_ARG_VALUE('name', name);
-    if (options !== undefined) {
+    if (options != null) {
       validateObject(options, 'options');
     }
     const startTime = options?.startTime ?? now();
@@ -86,8 +84,6 @@ class PerformanceMark extends PerformanceEntry {
     markTimings.set(name, startTime);
 
     let detail = options?.detail;
-    // The usage of != is intentional, we want to skip structuredClone
-    // for both undefined and null
     detail = detail != null ?
       structuredClone(detail) :
       null;