Re: [w3ctag/design-reviews] Incubation: Lightweight and Conditional Tracing for long animation frame timing API (Issue #1224)

xiaochengh left a comment (w3ctag/design-reviews#1224)

Hi @guohuideng2024, the TAG reviewed this and fount it much more plausible than the [previous approach](https://github.com/w3ctag/design-reviews/issues/1012). The overall mechanism looks highly feasible to us.

We have some comments at a lower level regarding the API design.

1. The proposed new APIs `markCondition()` and `measureConditional()` are directly tied to long animation frames, and therefore:
   - The connection between these APIs and LoAF is not clear from the names and may be confusing
   - This is not extendable to use cases like, e.g., we want to analyze long tasks instead of long animation frames, or if there are new types of performance incidents we want to introduce in the future
2. We are also unsure why we need to introduce "conditional" marks and measures, instead of somehow allowing existing marks and measures to be attributed to LoAF and similar incidents.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3ctag/design-reviews/issues/1224#issuecomment-4665223416
You are receiving this because you are subscribed to this thread.

Message ID: <w3ctag/design-reviews/issues/1224/4665223416@github.com>