Bug 175611

Summary:	[Payment Request] Some imported tests time out because they can't load echo-PaymentRequest.html
Product:	WebKit	Reporter:	Andy Estes <aestes>
Component:	Tools / Tests	Assignee:	Marcos Caceres <marcosc>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	aestes, ben_schwartz, cdumez, lforschler, marcosc, marcos, webkit-bug-importer, youennf
Priority:	P2	Keywords:	InRadar
Version:	WebKit Local Build
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=175612 https://bugs.webkit.org/show_bug.cgi?id=218308

Description Andy Estes 2017-08-15 18:07:43 PDT

Tests that time out:

imported/w3c/web-platform-tests/payment-request/allowpaymentrequest/allowpaymentrequest-attribute-cross-origin-bc-containers.https.html
imported/w3c/web-platform-tests/payment-request/allowpaymentrequest/no-attribute-cross-origin-bc-containers.https.html
imported/w3c/web-platform-tests/payment-request/allowpaymentrequest/removing-allowpaymentrequest.https.sub.html
imported/w3c/web-platform-tests/payment-request/allowpaymentrequest/setting-allowpaymentrequest-timing.https.sub.html
imported/w3c/web-platform-tests/payment-request/allowpaymentrequest/setting-allowpaymentrequest.https.sub.html

Comment 1 Chris Dumez 2020-10-28 15:34:25 PDT

Blocked access to external URL https://www1.localhost:9443/payment-request/allowpaymentrequest/echo-PaymentRequest.html?embed

This is the typical case of WPT using https://www1.localhost:9443 host, which we don't support. I believe this is a way to write those tests so that they use a host we support. Youenn knows how to fix that.

Comment 2 Marcos Caceres 2021-06-23 16:32:26 PDT

If anyone can point me to an example that works for Webkit, I'm happy to update the tests.

Comment 3 Radar WebKit Bug Importer 2024-03-28 19:34:46 PDT

<rdar://problem/125592721>

Comment 4 Marcos Caceres 2024-03-28 21:14:20 PDT

Pull request: https://github.com/WebKit/WebKit/pull/26603

Comment 5 EWS 2024-04-02 05:18:02 PDT

Committed 276932@main (339e8f017c40): <https://commits.webkit.org/276932@main>

Reviewed commits have been landed. Closing PR #26603 and removing active labels.