Bug 218063

Summary:	REGRESSION: [ Catalina ] 50+ webrtc test crashes on Catalina Debug
Product:	WebKit	Reporter:	Truitt Savell <tsavell>
Component:	WebRTC	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED CONFIGURATION CHANGED
Severity:	Normal	CC:	ap, eric.carlson, peng.liu6, webkit-bot-watchers-bugzilla, webkit-bug-importer, youennf
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Truitt Savell 2020-10-21 16:58:33 PDT

After a commit around r268758 50+ webrtc tests began crashing on Catalina WK2 Debug and around 20 on WK1 Debug. These tests are flaky and we will see runs with 50+ crashes and runs with 1 failure. 

history of runs:
https://build.webkit.org/builders/Apple-Catalina-Debug-WK2-Tests?numbuilds=50

Example:
https://build.webkit.org/builders/Apple-Catalina-Debug-WK2-Tests/builds/7339

This is reproducible on the bots, I am looking to find a test list that can reproduce these to be more specific.

Results:
https://build.webkit.org/results/Apple-Catalina-Debug-WK2-Tests/r268822%20(7339)/results.html

Comment 1 Radar WebKit Bug Importer 2020-10-21 16:59:13 PDT

<rdar://problem/70551805>

Comment 2 Alexey Proskuryakov 2020-10-21 19:15:35 PDT

# Fatal error in: Source/webrtc/rtc_base/operations_chain.h, line 136
# last system error: 0
# Check failed: (&sequence_checker_)->IsCurrent()

It would seem like r268758 only changes GPU Process code, which I think is not supposed to be on in tests. But there are no other potential culprits anywhere near.

Comment 3 Truitt Savell 2020-10-23 13:57:08 PDT

Speculatively reverted https://trac.webkit.org/changeset/268938/webkit to fix this. Bisection seems to have shown this as the regression point though the commit seems very unrelated

Comment 4 Ryan Haddad 2020-10-27 10:15:48 PDT

(In reply to Truitt Savell from comment #3)
> Speculatively reverted https://trac.webkit.org/changeset/268938/webkit to
> fix this. Bisection seems to have shown this as the regression point though
> the commit seems very unrelated
The issue persists, so this commit was not the cause of the crashes.

Comment 5 youenn fablet 2021-07-06 04:37:04 PDT

Marking as configuration changed, please reopen if crash is still happenning.