Skip to content

Commit

Permalink
benchmark: merge url.js with url-resolve.js
Browse files Browse the repository at this point in the history
url.js was broken since it didn't use the common.js runner. This fixes
that issue by merging it with url-resolve.js, which also benchmarks
url.resolve.

PR-URL: #5177
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
  • Loading branch information
AndreasMadsen authored and Myles Borins committed Jun 24, 2016
1 parent 921e856 commit 87cdb83
Showing 1 changed file with 25 additions and 12 deletions.
37 changes: 25 additions & 12 deletions benchmark/url/url-resolve.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,30 +3,43 @@ var common = require('../common.js');
var url = require('url');
var v8 = require('v8');

var hrefs = [
'http://example.com/',
'http://nodejs.org/docs/latest/api/url.html#url_url_format_urlobj',
'http://blog.nodejs.org/',
'https://encrypted.google.com/search?q=url&q=site:npmjs.org&hl=en',
'javascript:alert("node is awesome");',
'some.ran/dom/url.thing?oh=yes#whoo'
];


var paths = [
'../../../../../etc/passwd',
'../foo/bar?baz=boom',
'foo/bar',
'http://nodejs.org',
'./foo/bar?baz'
];

var bench = common.createBenchmark(main, {
type: ['one'],
n: [1e5],
href: Object.keys(hrefs),
path: Object.keys(paths),
n: [1e5]
});

function main(conf) {
var type = conf.type;
var n = conf.n | 0;

var inputs = {
one: ['http://example.com/', '../../../../../etc/passwd'],
};
var input = inputs[type] || [];
var href = hrefs[conf.href];
var path = paths[conf.path];

// Force-optimize url.resolve() so that the benchmark doesn't get
// disrupted by the optimizer kicking in halfway through.
for (var name in inputs)
url.resolve(inputs[name][0], inputs[name][1]);

url.resolve(href, path);
v8.setFlagsFromString('--allow_natives_syntax');
eval('%OptimizeFunctionOnNextCall(url.resolve)');

bench.start();
for (var i = 0; i < n; i += 1)
url.resolve(input[0], input[1]);
url.resolve(href, path);
bench.end(n);
}

0 comments on commit 87cdb83

Please sign in to comment.