More strict check for init["baseURL"] in URLPatternInit processing

[sisidovski]

init["baseURL"] is initialized with null, and map-exists only checks if there exists an entry with the key. This commit adds null or empty string check for init["baseURL"] in URLPatternInit processing.

• At least two implementers are interested (and none opposed):
  • Google Chrome
  • …
• Tests are written and can be reviewed and commented upon at:
  • n/a (this is a fix to how the spec expresses this, but does not represent a behavior change, and existing tests cover this behavior)
• Implementation bugs are filed:
  • Chromium: n/a (believed to work correctly in Chromium)
  • Gecko: https://bugzilla.mozilla.org/show_bug.cgi?id=1731418 (vendor does not yet implement the spec)
  • WebKit: [no known URLPattern bug] (vendor does not yet implement the spec)
  • Deno: n/a (no reason to believe a change is required)
  • kenchris/urlpattern-polyfill: n/a (no reason to believe a change is required)
• MDN issue is filed: n/a (change is on a spec detail not expressly documented)
• The top of this comment includes a clear commit message to use.

(See WHATWG Working Mode: Changes for more details.)

---

Preview | Diff

[sisidovski]

This seems to be unnecessary, closing.