RESOLVED FIXED224657
[GLIB] imported/w3c/web-platform-tests/service-workers/service-worker/next-hop-protocol.https.html is failing 
https://bugs.webkit.org/show_bug.cgi?id=224657
Summary [GLIB] imported/w3c/web-platform-tests/service-workers/service-worker/next-ho...
Diego Pino
Reported 2021-04-16 00:40:01 PDT
The test was removed in r276009 as it was solved. However, r276009 was partially reverted in r276021 and since then this test is failing again. The test should be reintroduced in glib TestExpectations. https://build.webkit.org/results/GTK-Linux-64-bit-Release-Tests/r276100%20%281194%29/imported/w3c/web-platform-tests/service-workers/service-worker/next-hop-protocol.https-diff.txt --- /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/next-hop-protocol.https-expected.txt +++ /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/next-hop-protocol.https-actual.txt @@ -1,4 +1,4 @@ -FAIL nextHopProtocol reports H1 correctly when routed via a service worker. assert_equals: nextHopProtocol is set on pass-through expected "http/1.1" but got "" +FAIL nextHopProtocol reports H1 correctly when routed via a service worker. assert_equals: nextHopProtocol is set on cached response expected "http/1.1" but got "" FAIL nextHopProtocol reports H2 correctly when routed via a service worker. assert_equals: nextHopProtocol is set on fallback expected "h2" but got "http/1.1"
Attachments
Add attachment proposed patch, testcase, etc.
Diego Pino
Comment 1 2021-04-16 00:49:21 PDT
Reintroduced test failure in r276122.
Radar WebKit Bug Importer
Comment 2 2021-04-23 01:10:24 PDT
<rdar://problem/77061210>
Carlos Garcia Campos
Comment 3 2021-06-03 05:19:51 PDT
I don't understand how this could work before or why it works in mac, because ServiceWorkerFetchTask::didFinish() always passes an empty metrics to Messages::WebResourceLoader::DidFinishResourceLoad(). Any idea Alex?
Diego Pino
Comment 4 2022-02-07 21:32:19 PST
Fixed by r288088. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.