Summary: | Test landed flaky: [iOS] TestWebKitAPI.WebKit2.CrashGPUProcessWhileCapturing | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||||
Component: | Tools / Tests | Assignee: | youenn fablet <youennf> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric.carlson, ews-watchlist, glenn, jer.noble, philipj, sergio, webkit-bot-watchers-bugzilla, webkit-bug-importer, youennf | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Other | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=221086 | ||||||||
Bug Depends on: | 221747 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Aakash Jain
2021-02-03 07:22:40 PST
Seems like it has been flaky since it was added 5 days back in https://trac.webkit.org/r272050 From https://ews-build.webkit.org/#/builders/9/builds/39710/steps/10/logs/stdio: TestWebKitAPI.WebKit2.CrashGPUProcessWhileCapturing (webrtc_video_engine.cc:3316): Absent receive stream; ignoring clearing encoded frame sink for ssrc 0 /Volumes/Data/worker/iOS-14-Simulator-Build-EWS/build/Tools/TestWebKitAPI/Tests/WebKit/GetUserMedia.mm:74 Expected equality of these values: @"PASS" Which is: "PASS" [message body] Which is: "FAIL checkVideoStatus" Created attachment 419264 [details]
Patch
Committed r272381: <https://trac.webkit.org/changeset/272381> All reviewed patches have been landed. Closing bug and clearing flags on attachment 419264 [details]. Reopening since flakiness continues in the bots. Do we think this is a test only issue, or should we revert the original change? (In reply to Ryan Haddad from comment #7) > Do we think this is a test only issue, or should we revert the original > change? This test identifies an issue in the way we handle GPU process capture on iOS. I am working on fixing this. Once done, the test should no longer be flaky. I hope to fix this by end of this week, early next week. I am fine skipping this test on iOS for now. The original change is fine. (In reply to youenn fablet from comment #8) > (In reply to Ryan Haddad from comment #7) > > Do we think this is a test only issue, or should we revert the original > > change? > > This test identifies an issue in the way we handle GPU process capture on > iOS. > I am working on fixing this. Once done, the test should no longer be flaky. Fix in https://bugs.webkit.org/show_bug.cgi?id=221747 Created attachment 420105 [details]
Patch
Committed r272802: <https://commits.webkit.org/r272802> All reviewed patches have been landed. Closing bug and clearing flags on attachment 420105 [details]. |