Bug 236678

Summary: REGRESSION(r289495): [ Monterey arm64 ] model-element/model-element-ready.html is a flaky assertion failure
Product: WebKit Reporter: Dawn Morningstar <Morningstar>
Component: New BugsAssignee: Dawn Morningstar <Morningstar>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, graouts, tsavell, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Dawn Morningstar
Reported 2022-02-15 16:20:27 PST
model-element/model-element-ready.html Is a flaky failure on Monterey on arm64. Failure appears to be introduced with changes in: https://trac.webkit.org/changeset/289495/webkit Revision r289495 affects CAFenceHandle. HISTORY: https://results.webkit.org/?suite=layout-tests&test=model-element%2Fmodel-element-ready.html&platform=mac&version_name=Monterey CRASH-LOG: No crash log found for WebKitTestRunner:55595. stdout: stderr: Assertion failed: ([self isInvalidated]), function -[CAFenceHandle dealloc], file CAFenceHandle.mm, line 112.
Attachments
Patch (1.43 KB, patch)
2022-02-15 16:41 PST, Dawn Morningstar
no flags
Dawn Morningstar
Comment 1 2022-02-15 16:21:19 PST
Crash was unable to be reproduced locally.
Radar WebKit Bug Importer
Comment 2 2022-02-15 16:22:52 PST
Dawn Morningstar
Comment 3 2022-02-15 16:41:14 PST
Truitt Savell
Comment 4 2022-02-15 16:44:34 PST
Comment on attachment 452108 [details] Patch Clearing flags on attachment: 452108 Committed r289864 (247304@trunk): <https://commits.webkit.org/247304@trunk>
Ryan Haddad
Comment 5 2022-05-19 14:48:53 PDT
Doesn't look like this test has been crashing lately. We should remove the flaky crash expectation.
Dawn Morningstar
Comment 6 2022-06-14 10:49:24 PDT
Re-opening for pull request https://github.com/WebKit/WebKit/pull/1509
EWS
Comment 7 2022-06-14 10:51:58 PDT
Test gardening commit r295527 (251532@main): <https://commits.webkit.org/251532@main> Reviewed commits have been landed. Closing PR #1509 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.