Bug 254389

Summary:	Document Quirks.cpp with original bugs
Product:	WebKit	Reporter:	Karl Dubost <karlcow>
Component:	New Bugs	Assignee:	Karl Dubost <karlcow>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Karl Dubost

Reported 2023-03-23 19:55:43 PDT

Currently, it is difficult to find the source of a quirk. Blame doesn't always give the source of the issue (specifically after code rewriting). It makes it hard for engineers who want to remove a quirk to go back in the history and find the origin and/or steps to reproduce the issue. The intent is to document with comments in the source code.

Attachments
Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2023-03-23 19:56:16 PDT

<rdar://problem/107167633>

Karl Dubost

Comment 2 2023-03-23 20:03:51 PDT

Pull request: https://github.com/WebKit/WebKit/pull/11901

EWS

Comment 3 2023-03-28 08:52:02 PDT

Committed 262216@main (afadc0ae8b64): <https://commits.webkit.org/262216@main> Reviewed commits have been landed. Closing PR #11901 and removing active labels.

Note You need to log in before you can comment on or make changes to this bug.