Bug 207894

Summary: [ Mac wk2 ] http/wpt/beacon/beacon-quota.html is flaky failing
Product: WebKit Reporter: Jason Lawrence <Lawrence.j>
Component: WebCore Misc.Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, cdumez, commit-queue, ggaren, webkit-bot-watchers-bugzilla, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Mac   
OS: macOS 10.14   
Attachments:
Description Flags
Patch none

Description Jason Lawrence 2020-02-18 09:00:33 PST
http/wpt/beacon/beacon-quota.html

Description:
This test is flaky failing on Mac wk2. The oldest failure that I am able to gather a diff from is r255116, but I see failures back to r245991.

History:
https://results.webkit.org/?suite=layout-tests&test=http%2Fwpt%2Fbeacon%2Fbeacon-quota.html&platform=mac&limit=50000


Diff:
--- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/http/wpt/beacon/beacon-quota-expected.txt
+++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/http/wpt/beacon/beacon-quota-actual.txt
@@ -1,6 +1,7 @@
 CONSOLE MESSAGE: line 44: Beacon API cannot load http://localhost:8800/. Reached maximum amount of queued data of 64Kb for keepalive requests
 CONSOLE MESSAGE: line 52: Beacon API cannot load http://localhost:8800/. Reached maximum amount of queued data of 64Kb for keepalive requests
+CONSOLE MESSAGE: line 54: Beacon API cannot load http://localhost:8800/WebKit/beacon/resources/beacon-preflight.py?allowCors=1&cmd=put&id=40ff52a3-4794-4fca-9db5-3ea23fdfb944. Reached maximum amount of queued data of 64Kb for keepalive requests
 
 PASS Beacon with a body above the Quota Limit should fail. 
-PASS Multiple Beacons Quota Limit 
+FAIL Multiple Beacons Quota Limit assert_true: Allocated quota should be returned once the beacon is no longer in flight expected true got false
Comment 1 Radar WebKit Bug Importer 2020-02-18 09:00:53 PST
<rdar://problem/59551688>
Comment 2 Jason Lawrence 2020-02-18 09:06:51 PST
I have marked this test as failing while this issue is investigated.
https://trac.webkit.org/changeset/256832/webkit
Comment 3 Jason Lawrence 2020-02-18 14:54:26 PST
I am able to reproduce this issue on r256844 with the command below.
run-webkit-tests http/wpt/beacon/beacon-quota.html --iterations 3000 --exit-after-n-failures 5 -f --force


[39/3000] http/wpt/beacon/beacon-quota.html failed unexpectedly (text diff)
[64/3000] http/wpt/beacon/beacon-quota.html failed unexpectedly (text diff)
[107/3000] http/wpt/beacon/beacon-quota.html failed unexpectedly (text diff)
[122/3000] http/wpt/beacon/beacon-quota.html failed unexpectedly (text diff)
[127/3000] http/wpt/beacon/beacon-quota.html failed unexpectedly (text diff)
Exiting early after 5 failures. 124 tests run.

119 tests ran as expected, 5 didn't (2876 didn't run):
Comment 4 Chris Dumez 2020-03-18 09:22:21 PDT
Created attachment 393869 [details]
Patch
Comment 5 Geoffrey Garen 2020-03-18 10:03:06 PDT
Comment on attachment 393869 [details]
Patch

r=me
Comment 6 WebKit Commit Bot 2020-03-18 10:57:50 PDT
Comment on attachment 393869 [details]
Patch

Clearing flags on attachment: 393869

Committed r258649: <https://trac.webkit.org/changeset/258649>
Comment 7 WebKit Commit Bot 2020-03-18 10:57:52 PDT
All reviewed patches have been landed.  Closing bug.