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

Description Dawn Morningstar 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.
Comment 1 Dawn Morningstar 2022-02-15 16:21:19 PST
Crash was unable to be reproduced locally.
Comment 2 Radar WebKit Bug Importer 2022-02-15 16:22:52 PST
<rdar://problem/88993883>
Comment 3 Dawn Morningstar 2022-02-15 16:41:14 PST
Created attachment 452108 [details]
Patch
Comment 4 Truitt Savell 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>
Comment 5 Ryan Haddad 2022-05-19 14:48:53 PDT
Doesn't look like this test has been crashing lately. We should remove the flaky crash expectation.
Comment 6 Dawn Morningstar 2022-06-14 10:49:24 PDT
Re-opening for pull request https://github.com/WebKit/WebKit/pull/1509
Comment 7 EWS 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.