Skip to content

Fetch latest timeline events before exporting Performance data - #10055

Draft
khanak0509 wants to merge 1 commit into
flutter:masterfrom
khanak0509:fix-9731-export-timeline-events
Draft

khanak0509 wants to merge 1 commit into
flutter:masterfrom
khanak0509:fix-9731-export-timeline-events

Conversation

@khanak0509

@khanak0509 khanak0509 commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #9731

this PR fixes issue #9731 where data exported from the Performance screen was missing the most recent timeline events. Timeline events are only pulled from the VM every 10 seconds, so frames drawn after the last poll were exported without them. Now the latest timeline events are pulled before exporting, the same way the Network screen does with fetchFullDataBeforeExport.

Before

most recent frames (1331–1411) had no timeline events in the file.

Screenshot 2026-10-20 at 2 00 34 AM

After

all of them had timeline events, including the most recent frame

Screenshot 2026-10-20 at 2 02 41 AM

Pre-launch Checklist

General checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I updated/added relevant documentation (doc comments with ///).

Issues checklist

Tests checklist

  • I added new tests to check the change I am making...
  • OR there is a reason for not adding tests, which I explained in the PR description.

AI-tooling checklist

  • I did not use any AI tooling in creating this PR.
  • OR I did use AI tooling, and...
    • I read the AI contributions guidelines and agree to follow them.
    • I reviewed all AI-generated code before opening this PR.
    • I understand and am able to discuss the code in this PR.
    • I have verifed the accuracy of any AI-generated text included in the PR description.
    • I commit to verifying the accuracy of any AI-generated code or text that I upload in response to review comments.

Feature-change checklist

  • This PR does not change the DevTools UI or behavior and...
    • I added the release-notes-not-required label or left a comment requesting the label be added.
  • OR this PR does change the DevTools UI or behavior and...
    • I added an entry to packages/devtools_app/release_notes/NEXT_RELEASE_NOTES.md.
    • I included before/after screenshots and/or a GIF demo of the new UI to my PR description.
    • I ran the DevTools app locally to manually verify my changes.

build.yaml badge

If you need help, consider asking for help on Discord.

This branch has not been deployed

No deployments
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.

Missing timeline events in performance data export

1 participant