Skip to content

Move over an explainer to MSEdge repo so I can proceed to next step.#1301

Open
guohuideng2024 wants to merge 6 commits intoMicrosoftEdge:mainfrom
guohuideng2024:main
Open

Move over an explainer to MSEdge repo so I can proceed to next step.#1301
guohuideng2024 wants to merge 6 commits intoMicrosoftEdge:mainfrom
guohuideng2024:main

Conversation

@guohuideng2024
Copy link
Copy Markdown
Member

No description provided.

@guohuideng2024 guohuideng2024 requested a review from Copilot April 14, 2026 20:18
@guohuideng2024
Copy link
Copy Markdown
Member Author

I named the explainer "explainer-for-loaf.md" because I expect more explainers could be added into this folder in future, and the future explainer can be named as "explainer-for-event-timing.md" etc

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a “Lightweight and Conditional Tracing” explainer document describing an extension to User Timing aimed at capturing conditional marks/measures for LoAF investigations.

Changes:

  • Introduces a new explainer Markdown document for conditional tracing tied to LoAF.
  • Documents proposed API surface (markConditional / measureConditional) and includes example usage.
  • Adds initial Security/Privacy questionnaire responses for the proposed feature.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ConditionalTracing/explainer-for-loaf.md
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
@guohuideng2024
Copy link
Copy Markdown
Member Author

@sfortiner : The explainer has already been through WG discussion and passed. I am moving it here to comply MS requirement that the explainer must be in this repo.

Thanks for reviewing.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an explainer document for a proposed “Lightweight and Conditional Tracing” mechanism intended to help developers attribute Long Animation Frames (LoAF) more precisely, by recording user-timing-like marks/measures only when a relevant performance incident occurs.

Changes:

  • Introduce markConditional() / measureConditional() API concept and expected reporting surface (userTimingEntries on PerformanceLongAnimationFrameTiming).
  • Provide sample JavaScript usage with PerformanceObserver and example outputs.
  • Include initial alternatives discussion and Security/Privacy self-review questionnaire responses.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md Outdated
Comment thread ConditionalTracing/explainer-for-loaf.md
Comment thread ConditionalTracing/explainer-for-loaf.md
guohuideng2024 and others added 2 commits April 15, 2026 17:05
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@aluhrs13 aluhrs13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guohuideng2024 - Can you add this to the README too please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants