url-polyfill is loaded in Safari when not required

[nekodex]

Seems to be because Safari escapes the pathname set by url-polyfill.js#L173, causing the comparison (u.href === 'http://a/c%20d') to return false:

[supperchong]

I find the same problem.When I use url-polyfill and pdfjs-dist in my web,the url-polyfill will lose efficacy. Because pdf-dist use this code to check if URL is valid.
if (typeof URL === 'function' && _typeof(URL.prototype) === 'object' && 'origin' in URL.prototype) { var u = new URL('b', 'http://a'); u.pathname = 'c%20d'; hasWorkingUrl = u.href === 'http://a/c%20d'; }
But in safari,
u.href==='http://a/c%2520d'
And then the pdfjs-dist will rewrite URL.But the module doesn't achieve the searchParams method.fortunately the pdfjs-dist have Stop exposing the URL polyfill in the global scope.

More over ,If this is a bug ? url-polyfill use this code to check URL is supported.But after execute the module .The check is false also in safari.
var checkIfURLIsSupported = function() { try { var u = new URL('b', 'http://a'); u.pathname = 'c%20d'; return (u.href === 'http://a/c%20d') && u.searchParams; } catch(e) { return false; } };

[lifaon74]

I don't have Safari on the hand. Could you fix it on your side, run the tests and create a pull request please ?

[moshest]

Currently looks like everything works fine (Safari 12.1.1):

[janicklas-ralph]

This is still an issue in Safari 10. Any idea what should be the fix?

[lifaon74]

@janicklas-ralph I dont have any Safari on the hand, could you please fork the project, fix the bug, and then do a pull request ?

[lifaon74]

Ok, version 1.1.8 released. Tell if everything is OK