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 Bugs

Assignee:

Dawn Morningstar <Morningstar>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

dino, graouts, tsavell, webkit-bot-watchers-bugzilla, webkit-bug-importer

Priority:

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.