wpt icon indicating copy to clipboard operation
wpt copied to clipboard

Pass the right arguments to make FedCM tests work

Open cbiesinger opened this issue 2 years ago • 3 comments

This effectively ports these Chromium patches to wptrunner: https://chromium-review.googlesource.com/c/chromium/src/+/3671447/3/content/web_test/browser/web_test_browser_main_runner.cc https://chromium-review.googlesource.com/c/chromium/src/+/3546261

We need to map :443 to the WPT port because of the root manifest part of the FedCM spec: https://fedidcg.github.io/FedCM/#check-the-root-manifest

cbiesinger avatar Jul 29 '22 18:07 cbiesinger

So AFAICT this is defacto adding a new wpt server that's on port 443, but only accessible to Chrome? If that understanding is correct we can't do this.

Would it work to make this use the recently added PAC support?

jgraham avatar Aug 02 '22 10:08 jgraham

Oh, I didn't realize PAC support was added. Can you point me to the code?

cbiesinger avatar Aug 02 '22 15:08 cbiesinger

(I've since found ddeae538bbc9d634c983238fceea89ff08d00a86 and am working on updating this PR now)

cbiesinger avatar Aug 03 '22 18:08 cbiesinger

Closing this, I'll do this differently

cbiesinger avatar Jun 21 '23 19:06 cbiesinger