url: remove javascript URL special case

PR-URL: #12507
Fixes: #11485
Reviewed-By: James M Snell <[email protected]>

Author: watilde

lib/internal/url.js

@@ -556,8 +556,6 @@ Object.defineProperties(URL.prototype, {
       const ctx = this[context];
       // toUSVString is not needed.
       hash = `${hash}`;
-      if (this.protocol === 'javascript:')
-        return;
       if (!hash) {
         ctx.fragment = null;
         ctx.flags &= ~binding.URL_FLAGS_HAS_FRAGMENT;

test/fixtures/url-setter-tests.js

@@ -1800,13 +1800,13 @@ module.exports =
                 "hash": "#%c3%89t%C3%A9"
             }
         },
-        // {
-        //     "href": "javascript:alert(1)",
-        //     "new_value": "castle",
-        //     "expected": {
-        //         "href": "javascript:alert(1)#castle",
-        //         "hash": "#castle"
-        //     }
-        // }
+        {
+            "href": "javascript:alert(1)",
+            "new_value": "castle",
+            "expected": {
+                "href": "javascript:alert(1)#castle",
+                "hash": "#castle"
+            }
+        }
     ]
 }