fix: remove IE8 url parsing workaround

[gkatsev]

No description provided.

[codecov]

Codecov Report

Merging #7334 (9def53c) into main (bba6e17) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #7334      +/-   ##
==========================================
+ Coverage   79.53%   79.57%   +0.04%     
==========================================
  Files         115      115              
  Lines        7270     7262       -8     
  Branches     1747     1745       -2     
==========================================
- Hits         5782     5779       -3     
+ Misses       1488     1483       -5     

Impacted Files	Coverage Δ
src/js/utils/url.js	78.04% <100.00%> (+8.66%)	⬆️
src/js/tracks/text-track.js	91.09% <0.00%> (-0.69%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bba6e17...9def53c. Read the comment docs.

[gkatsev]

Fun, IE11 fails without this in some cases. Making changing how the removed code works causes it to fail to work as well. So, needs more thought here.

[gkatsev]

codecov thinks this isn't covered, but it gets covered in IE11. We just only run coverage in chrome.

[brandonocasey]

I think there is a special comment you can use to skip this if statement.

I think it's /* istanbul ignore if */ from https://github.com/gotwarlost/istanbul/blob/master/ignoring-code-for-coverage.md