benchmark: differentiate whatwg and legacy url

benchmark/url/legacy-url-get-prop.js

@@ -0,0 +1,42 @@
+'use strict';
+const common = require('../common.js');
+const url = require('url');
+const assert = require('assert');
+
+const bench = common.createBenchmark(main, {
+  type: common.urlDataTypes,
+  method: ['legacy', 'whatwg'],
+  e: [1],
+});
+
+function main({ type, e }) {
+  const data = common.bakeUrlData(type, e, false, true);
+  const obj = url.parse(data[0]);
+  const noDead = {
+    protocol: obj.protocol,
+    auth: obj.auth,
+    host: obj.host,
+    hostname: obj.hostname,
+    port: obj.port,
+    pathname: obj.pathname,
+    search: obj.search,
+    hash: obj.hash,
+  };
+  const len = data.length;
+  // It's necessary to assign the values to an object
+  // to avoid loop invariant code motion.
+  bench.start();
+  for (let i = 0; i < len; i++) {
+    const obj = data[i];
+    noDead.protocol = obj.protocol;
+    noDead.auth = obj.auth;
+    noDead.host = obj.host;
+    noDead.hostname = obj.hostname;
+    noDead.port = obj.port;
+    noDead.pathname = obj.pathname;
+    noDead.search = obj.search;
+    noDead.hash = obj.hash;
+  }
+  bench.end(len);
+  assert.ok(noDead);
+}

benchmark/url/legacy-url-parse.js

@@ -0,0 +1,24 @@
+'use strict';
+const common = require('../common.js');
+const url = require('url');
+const assert = require('assert');
+
+const bench = common.createBenchmark(main, {
+  withBase: ['true', 'false'],
+  type: common.urlDataTypes,
+  e: [1],
+  method: ['legacy', 'whatwg'],
+});
+
+function main({ e, type }) {
+  const data = common.bakeUrlData(type, e, false, false);
+  let result = url.parse(data[0]);  // Avoid dead code elimination
+
+  bench.start();
+  for (let i = 0; i < data.length; ++i) {
+    result = url.parse(data[i]);
+  }
+  bench.end(data.length);
+
+  assert.ok(result);
+}

benchmark/url/whatwg-url-get-prop.js

@@ -0,0 +1,40 @@
+'use strict';
+const common = require('../common.js');
+const url = require('url');
+const URL = url.URL;
+const assert = require('assert');
+
+const bench = common.createBenchmark(main, {
+  type: common.urlDataTypes,
+  e: [1],
+});
+
+function main({ type, e }) {
+  const data = common.bakeUrlData(type, e, false, true);
+  const obj = new URL(data[0]);
+  const noDead = {
+    protocol: obj.protocol,
+    auth: `${obj.username}:${obj.password}`,
+    host: obj.host,
+    hostname: obj.hostname,
+    port: obj.port,
+    pathname: obj.pathname,
+    search: obj.search,
+    hash: obj.hash,
+  };
+  const len = data.length;
+  bench.start();
+  for (let i = 0; i < len; i++) {
+    const obj = data[i];
+    noDead.protocol = obj.protocol;
+    noDead.auth = `${obj.username}:${obj.password}`;
+    noDead.host = obj.host;
+    noDead.hostname = obj.hostname;
+    noDead.port = obj.port;
+    noDead.pathname = obj.pathname;
+    noDead.search = obj.search;
+    noDead.hash = obj.hash;
+  }
+  bench.end(len);
+  assert.ok(noDead);
+}

benchmark/url/legacy-vs-whatwg-url-parse.js → benchmark/url/whatwg-url-parse.js

@@ -8,20 +8,8 @@ const bench = common.createBenchmark(main, {
   withBase: ['true', 'false'],
   type: common.urlDataTypes,
   e: [1],
-  method: ['legacy', 'whatwg'],
 });
 
-function useLegacy(data) {
-  const len = data.length;
-  let result = url.parse(data[0]);  // Avoid dead code elimination
-  bench.start();
-  for (let i = 0; i < len; ++i) {
-    result = url.parse(data[i]);
-  }
-  bench.end(len);
-  return result;
-}
-
 function useWHATWGWithBase(data) {
   const len = data.length;
   let result = new URL(data[0][0], data[0][1]);  // Avoid dead code elimination
@@ -45,22 +33,10 @@ function useWHATWGWithoutBase(data) {
   return result;
 }
 
-function main({ e, method, type, withBase }) {
+function main({ e, type, withBase }) {
   withBase = withBase === 'true';
-  let noDead;  // Avoid dead code elimination.
-  let data;
-  switch (method) {
-    case 'legacy':
-      data = common.bakeUrlData(type, e, false, false);
-      noDead = useLegacy(data);
-      break;
-    case 'whatwg':
-      data = common.bakeUrlData(type, e, withBase, false);
-      noDead = withBase ? useWHATWGWithBase(data) : useWHATWGWithoutBase(data);
-      break;
-    default:
-      throw new Error(`Unknown method ${method}`);
-  }
+  const data = common.bakeUrlData(type, e, withBase === 'true', false);
+  const noDead = withBase ? useWHATWGWithBase(data) : useWHATWGWithoutBase(data);
 
   assert.ok(noDead);
 }