fix block of protocol change for file with empty host in the URL polyfill

Author: zloirock

CHANGELOG.md

@@ -34,6 +34,7 @@
 - Fixed some percent encode cases / character sets in the `URL` polyfill
 - Fixed some cases of `''` and `null` host handling in the `URL` polyfill
 - Fixed host parsing with `hostname = host:port` in the `URL` polyfill
+- Fixed block of protocol change for file with empty host in the `URL` polyfill
 - Fixed invalid code points handling in UTF-8 decode in the `URLSearchParams` polyfill
 - Fixed some cases of serialization in `URL` polyfill (`/.` prefix for non-special URLs with `null` host and path starting with empty segment)
 - Fixed `URL` polyfill `.origin` getter with `blob` scheme

packages/core-js/modules/web.url.constructor.js

@@ -602,6 +602,7 @@ URLState.prototype = {
 
         case FILE:
           url.scheme = 'file';
+          url.host = '';
           if (chr === '/' || chr === '\\') state = FILE_SLASH;
           else if (base && base.scheme === 'file') {
             switch (chr) {
@@ -624,8 +625,8 @@ URLState.prototype = {
                 state = FRAGMENT;
                 break;
               default:
+                url.host = base.host;
                 if (!startsWithWindowsDriveLetter(join(arraySlice(codePoints, pointer), ''))) {
-                  url.host = base.host;
                   url.path = arraySlice(base.path);
                   url.shortenPath();
                 }

tests/unit-global/web.url.js

@@ -42,6 +42,8 @@ QUnit.test('URL constructor', assert => {
   assert.same(String(new URL('http://0300.168.0xG0')), 'http://0300.168.0xg0/', 'incorrect IPv4 parsed as host');
 
   assert.same(String(new URL('file:///var/log/system.log')), 'file:///var/log/system.log', 'file scheme');
+  assert.same(String(new URL('file:foo')), 'file:///foo', 'file scheme without slashes');
+  assert.same(new URL('file:foo').host, '', 'file scheme without slashes: host');
   // assert.same(String(new URL('file://nnsc.nsf.net/bar/baz')), 'file://nnsc.nsf.net/bar/baz', 'file scheme'); // 'file:///bar/baz' in FF
   // assert.same(String(new URL('file://localhost/bar/baz')), 'file:///bar/baz', 'file scheme'); // 'file://localhost/bar/baz' in Chrome
 
@@ -199,6 +201,10 @@ QUnit.test('URL#protocol', assert => {
     assert.same(url.protocol, 'http:');
     assert.same(url.href, 'http://zloirock.ru/', 'incorrect scheme');
     assert.same(String(url), 'http://zloirock.ru/', 'incorrect scheme');
+
+    url = new URL('file:foo');
+    url.protocol = 'http:';
+    assert.same(url.protocol, 'file:', 'file with empty host: protocol change blocked');
   }
 });
 

tests/unit-pure/web.url.js

@@ -43,6 +43,8 @@ QUnit.test('URL constructor', assert => {
   assert.same(String(new URL('http://0300.168.0xG0')), 'http://0300.168.0xg0/', 'incorrect IPv4 parsed as host');
 
   assert.same(String(new URL('file:///var/log/system.log')), 'file:///var/log/system.log', 'file scheme');
+  assert.same(String(new URL('file:foo')), 'file:///foo', 'file scheme without slashes');
+  assert.same(new URL('file:foo').host, '', 'file scheme without slashes: host');
   // assert.same(String(new URL('file://nnsc.nsf.net/bar/baz')), 'file://nnsc.nsf.net/bar/baz', 'file scheme'); // 'file:///bar/baz' in FF
   // assert.same(String(new URL('file://localhost/bar/baz')), 'file:///bar/baz', 'file scheme'); // 'file://localhost/bar/baz' in Chrome
 
@@ -200,6 +202,10 @@ QUnit.test('URL#protocol', assert => {
     assert.same(url.protocol, 'http:');
     assert.same(url.href, 'http://zloirock.ru/', 'incorrect scheme');
     assert.same(String(url), 'http://zloirock.ru/', 'incorrect scheme');
+
+    url = new URL('file:foo');
+    url.protocol = 'http:';
+    assert.same(url.protocol, 'file:', 'file with empty host: protocol change blocked');
   }
 });