Bug 263348 - [SVG] Recursive marker bounding box computation is ad-hocly done
Summary: [SVG] Recursive marker bounding box computation is ad-hocly done
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-10-18 17:18 PDT by Yusuke Suzuki
Modified: 2023-10-25 17:19 PDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yusuke Suzuki 2023-10-18 17:18:45 PDT 
Right now, the spec and the implementation do not have clear view about how to handle circular / recursive marker bounding box computation.
As a result, we are right now just preventing the crash, and that's it. We are using stale repaint bounding box in a bit random way (if some gets re-layout, and some didn't get, then some renderer use the other renderer's stale repaint bounding box randomly).
We need reasonable behavior for this pattern. (including prohibiting this kind of SVG, but ultimately, it is nice if we can have clear description in the spec. Right now, the spec is not explaining the way to stop this recursion).
Comment 1 Yusuke Suzuki 2023-10-18 17:20:27 PDT 
Detailed discussion is in https://github.com/WebKit/WebKit/pull/19103's review comments. We agree that currently this is not specified and our implementation can grab stale repaint bounding box.
Comment 2 Radar WebKit Bug Importer 2023-10-25 17:19:23 PDT 
<rdar://problem/117505583>