videojs#3100 fix: moved protocol fix to other protocol related condit…

…ionals and expanded from (details.protocol === '') to (!details.protocol) Both implement suggestions from @misteroneill .
mmodrow · Oct 25, 2017 · e38a857 · e38a857
1 parent 1d99a2d
commit e38a857
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/js/utils/url.js b/src/js/utils/url.js
@@ -81,12 +81,12 @@ export const parseUrl = function(url) {
     details.host = details.host.replace(/:443$/, '');
   }
 
-  if (addToBody) {
-    document.body.removeChild(div);
+  if (!details.protocol) {
+    details.protocol = window.location.protocol;
   }
 
-  if (details.protocol === '') {
-    details.protocol = window.location.protocol;
+  if (addToBody) {
+    document.body.removeChild(div);
   }
 
   return details;