Skip to content

Commit

Permalink
benchmark: refactor whatwg-url-properties
Browse files Browse the repository at this point in the history
PR-URL: nodejs#11264
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
  • Loading branch information
joyeecheung authored and italoacasas committed Feb 16, 2017
1 parent b9bc94f commit bcc8111
Showing 1 changed file with 9 additions and 25 deletions.
34 changes: 9 additions & 25 deletions benchmark/url/whatwg-url-properties.js
Original file line number Diff line number Diff line change
@@ -1,16 +1,11 @@
'use strict';
const common = require('../common.js');
const URL = require('url').URL;
const inputs = require('../fixtures/url-inputs.js').urls;

var common = require('../common.js');
var URL = require('url').URL;

var bench = common.createBenchmark(main, {
url: [
'http://example.com/',
'https://encrypted.google.com/search?q=url&q=site:npmjs.org&hl=en',
'javascript:alert("node is awesome");',
'http://user:pass@foo.bar.com:21/aaa/zzz?l=24#test'
],
prop: ['toString', 'href', 'origin', 'protocol',
const bench = common.createBenchmark(main, {
input: Object.keys(inputs),
prop: ['href', 'origin', 'protocol',
'username', 'password', 'host', 'hostname', 'port',
'pathname', 'search', 'searchParams', 'hash'],
n: [1e4]
Expand All @@ -34,14 +29,6 @@ function get(n, url, prop) {
bench.end(n);
}

function stringify(n, url, prop) {
bench.start();
for (var i = 0; i < n; i += 1) {
url.toString();
}
bench.end(n);
}

const alternatives = {
href: 'http://user:pass@foo.bar.com:21/aaa/zzz?l=25#test',
protocol: 'https:',
Expand All @@ -61,7 +48,8 @@ function getAlternative(prop) {

function main(conf) {
const n = conf.n | 0;
const url = new URL(conf.url);
const input = inputs[conf.input];
const url = new URL(input);
const prop = conf.prop;

switch (prop) {
Expand All @@ -74,17 +62,13 @@ function main(conf) {
case 'pathname':
case 'search':
case 'hash':
case 'href':
setAndGet(n, url, prop, getAlternative(prop));
break;
// TODO: move href to the first group when the setter lands.
case 'href':
case 'origin':
case 'searchParams':
get(n, url, prop);
break;
case 'toString':
stringify(n, url);
break;
default:
throw new Error('Unknown prop');
}
Expand Down

0 comments on commit bcc8111

Please sign in to comment.