tree 5ee2b0b84d136efa8c236facae267f23dc323e79
parent 909c0dd89d4a2b43a1e6a3216b427f64556795e7
author Ben Kelly <wanderview@chromium.org> 1644361252 -0800
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1644361984 -0800

URLPattern: Set unmatched optional groups to undefined instead of ''.

This addresses the issues raised in:

https://github.com/WICG/urlpattern/issues/162

The main changes in this CL are:

1. The webidl is modified to allow passed back undefined.  I'm told
   real webidl should support `(USVString or undefined)` here, but our
   webidl compiler does not support that.  So we use `any` instead.
2. We must examine if the WTF::String is null or not before populating
   the returned array.  If its null, then we convert to undefined
   instead.

Bug: 1292699
Change-Id: I27a0be2567bb9ce4592ca15a5216fd5a58bdbf17
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3428631
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Ben Kelly <wanderview@chromium.org>
Cr-Commit-Position: refs/heads/main@{#968611}