Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@farchond
Copy link
Contributor

@farchond farchond commented May 8, 2020

Currently, we request a presentation time of 0 for all frames, thus relying on wakeup ordering between Flutter and Scenic to have reasonable behavior. This is unreliable at best, and necessitates coordination between Flutter and Scenic code changes in order to not regress performance.

This PR enables Flutter to instead target frame times based on information returned every vsync interval from Scenic. No change in performance is expected, as we're still targeting the same vsyncs, only now we're doing so explicitly.

Copy link
Contributor

@mikaelpessa mikaelpessa left a comment

Choose a reason for hiding this comment

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

Looks good to me

@farchond farchond force-pushed the robust_scheduling branch from 4a75565 to 0b62788 Compare June 2, 2020 20:15
Copy link
Contributor

@iskakaushik iskakaushik left a comment

Choose a reason for hiding this comment

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

LGTM

@farchond farchond force-pushed the robust_scheduling branch from 56231cd to a607ca8 Compare June 9, 2020 19:49
@iskakaushik iskakaushik merged commit 80230d8 into flutter:master Jun 9, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 10, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 10, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 10, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 10, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants