fix: removes localName assignment.

[gjanblaszczyk]

This PR is related to #15 and removes the localName (read-only) assignment.

[gjanblaszczyk]

I have created this PR because I have a problem with subtitles when I import video.js as an ES module in the browser. I see a similar PR on the official repo (mozilla/vtt.js#371) but it is still not merged. I wonder if we could merge the PR inside video.js fork to fix a problem. @gkatsev, any thoughts?

[gkatsev]

Thanks for pinging again, totally forgot about it. Unfortunately, mozilla/vtt.js is basically unmaintained right now
In Video.js for various reasons, we had just disabled "use strict" for our bundler, but that may not necessarily work in other bundlers.
I think this is probably reasonably, I don't imagine it'll cause many issues.

[gjanblaszczyk]

Yes, I think the same, I have done some tests for this PR and I haven't found any problems... It would be great to see it merged.

[gkatsev]

This may allow us to remove a change we did in Video.js to ask rollup not to add "use strict" to the top of modules.

[gjanblaszczyk]

Yeah, This is a great idea.