224657 2021-04-16 00:40:01 -0700 [GLIB] imported/w3c/web-platform-tests/service-workers/service-worker/next-hop-protocol.https.html is failing 2022-02-07 21:32:19 -0800 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 dpino webkit-unassigned achristensen bugs-noreply cgarcia webkit-bug-importer oldest_to_newest 1750968 0 dpino 2021-04-16 00:40:01 -0700 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" 1750970 1 dpino 2021-04-16 00:49:21 -0700 Reintroduced test failure in r276122. 1753490 2 webkit-bug-importer 2021-04-23 01:10:24 -0700 <rdar://problem/77061210> 1766349 3 cgarcia 2021-06-03 05:19:51 -0700 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? 1838524 4 dpino 2022-02-07 21:32:19 -0800 Fixed by r288088. Closing bug.