Add statics from builtin as needed about url

[eligrey]

Wouldn't exactly say sometimes. Every single current release of every major browser supports the API. Also, if by "add those" you mean save references to the methods and put them on a new URL object, that is not going to work either, as URL is also used as a native function for URL parsing. (new URL(url_here).origin etc.)

Simply put, URL is a global variable that should not ever be redefined unless you're polyfilling it, as I do in Blob.js. Whatever Polymer needs it for, it can use a differently named global variable if it's not using it for the sole purpose of polyfilling it.

[arv]

You are right, all browsers we care about have URL.createObjectURL and URL.revokeObjectURL.

We already feature detect and keep the original URL if it can be new'ed.

The PR makes sure that we add the statics if the builtin is not newable.

[eligrey]

Oh, so Polymer uses URL for it's URL processing abilities. Do you realize that document.createElement("a").href = url is an equivalent superset of new URL(url)?

See my new URL(...) polyfill for more information. You will notice that origin is manually handled, as some very old browsers don't support that natively.

[arv]

@eligrey That is mostly correct. There are some issues with it. And it is also slower, since it needs a new document and uses <base>. I took the <a> approach in arv/DOM-URL-Polyfill but we decided to go down the current route and base this on the reference implementation of the spec.

[eligrey]

Ah, thanks for linking me to that. I wasn't aware of that polyfill, mostly as mine focuses on the blob aspects.