Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embedder VsyncWaiter must schedule a frame for the right time #29408

Merged
merged 1 commit into from
Nov 1, 2021

Conversation

iskakaushik
Copy link
Contributor

@iskakaushik iskakaushik commented Oct 29, 2021

Prior to #28817 super class vsync waiter would post the tast to the future if the frame start time is at a later poit. This change made it so that the sub classes are responsible for posting the task at the right time.

This change makes it so that vsync waiter embedder respects that contract.

Fixes flutter/flutter#92603

Prior to flutter#28817 super class vsync
waiter would post the tast to the future if the frame start time is at a
later poit. This change made it so that the sub classes are responsible
for posting the task at the right time.

This change makes it so that vsync waiter embedder respects that
contract.

Fixes flutter/flutter#92603
@flutter-dashboard flutter-dashboard bot added the embedder Related to the embedder API label Oct 29, 2021
@google-cla google-cla bot added the cla: yes label Oct 29, 2021
@iskakaushik iskakaushik merged commit 43561d8 into flutter:master Nov 1, 2021
@iskakaushik iskakaushik deleted the issue-92603 branch November 1, 2021 17:34
godofredoc added a commit that referenced this pull request Nov 3, 2021
#29508)

* Embedder VsyncWaiter must schedule a frame for the right time (#29408)

* Ensure vsync callback completes before resetting the engine. (#29488)

Fixes: flutter/flutter#92934

Co-authored-by: Kaushik Iska <iska.kaushik@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes embedder Related to the embedder API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Embedder VSync Waiter needs to FireCallback at the Future timestamp
2 participants