Bug 208004

Summary: REGRESSION: (r257025) [ Mac wk2 ] http/tests/inspector/network/resource-sizes-disk-cache.html is failing.
Product: WebKit Reporter: Jason Lawrence <Lawrence.j>
Component: Web InspectorAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: inspector-bugzilla-changes, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Mac   
OS: macOS 10.15   
See Also: https://bugs.webkit.org/show_bug.cgi?id=207882

Description Jason Lawrence 2020-02-20 07:15:26 PST
http/tests/inspector/network/resource-sizes-disk-cache.html

Description:
This test is failing on Mac wk2. It appears to have a regression point of r257025.

History:
https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Finspector%2Fnetwork%2Fresource-sizes-disk-cache.html&limit=50000&platform=mac&flavor=wk2

Diff:
--- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/http/tests/inspector/network/resource-sizes-disk-cache-expected.txt
+++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/http/tests/inspector/network/resource-sizes-disk-cache-actual.txt
@@ -9,12 +9,18 @@
 PASS: compressed should be false.
 PASS: responseSource should be Symbol(disk-cache).
 size: 12940
-requestBodyTransferSize: 0
-responseBodyTransferSize: 0
+requestBodyTransferSize: NaN
+responseBodyTransferSize: NaN
 estimatedNetworkEncodedSize: 0
-networkEncodedSize: 0
+networkEncodedSize: NaN
 PASS: estimatedTotalTransferSize should be >= (encoded body size + headers).
-PASS: networkTotalTransferSize should be >= (encoded body size + headers).
-PASS: requestHeadersTransferSize should be empty.
-PASS: responseHeadersTransferSize should be empty.
+FAIL: networkTotalTransferSize should be >= (encoded body size + headers).
+    Expected: greater than or equal to 0
+    Actual: NaN
+FAIL: requestHeadersTransferSize should be empty.
+    Expected: 0
+    Actual: NaN
+FAIL: responseHeadersTransferSize should be empty.
+    Expected: 0
+    Actual: NaN
Comment 1 Radar WebKit Bug Importer 2020-02-20 07:16:00 PST
<rdar://problem/59628275>
Comment 2 Jason Lawrence 2020-02-20 07:26:47 PST
I have marked this test as failing while this issue is investigated.
https://trac.webkit.org/changeset/257057/webkit
Comment 3 Jason Lawrence 2020-02-20 07:29:35 PST
It looks like the regression point is r257025. I was able to reproduce the issue on r257052 and r257025, but was not able to reproduce it onr257013.
r257052:
run-webkit-tests http/tests/inspector/network/resource-sizes-disk-cache.html --iterations 4000 --exit-after-n-failures 5 -f —force[12/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[15/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[18/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[22/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[26/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
Exiting early after 5 failures. 23 tests run.
18 tests ran as expected, 5 didn't (3977 didn't run):

r257025:
run-webkit-tests http/tests/inspector/network/resource-sizes-disk-cache.html --iterations 4000 --exit-after-n-failures 5 -f --force
[12/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[14/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[18/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[21/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
[22/4000] http/tests/inspector/network/resource-sizes-disk-cache.html failed unexpectedly (text diff)
Exiting early after 5 failures. 19 tests run.
14 tests ran as expected, 5 didn't (3981 didn't run):

r257013:
run-webkit-tests http/tests/inspector/network/resource-sizes-disk-cache.html --iterations 4000 --exit-after-n-failures 5 -f --force
All 4000 tests ran as expected.
Comment 4 Jason Lawrence 2020-02-20 08:14:55 PST
I have rolled this commit out.
https://trac.webkit.org/changeset/257061/webkit
Comment 5 Jason Lawrence 2020-02-20 08:38:34 PST
I removed the test expectations.
https://trac.webkit.org/changeset/257063/webkit