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

plumb frame number through to framework #26233

Merged

Conversation

jonahwilliams
Copy link
Member

The engine side work of flutter/flutter#80735

This allows events sent by the framework to be easily matched with the corresponding engine side events.

@flutter-dashboard flutter-dashboard bot added the platform-web Code specifically for the web engine label May 18, 2021
@google-cla google-cla bot added the cla: yes label May 18, 2021
@@ -40,8 +40,14 @@ class FrameTiming {
return data_[phase] = value;
}

int64_t GetFrameNumber() const { return frame_number_; }
Copy link
Member Author

Choose a reason for hiding this comment

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

ahh this should probably be unsigned, will update

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

@jonahwilliams jonahwilliams added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label May 19, 2021
@fluttergithubbot fluttergithubbot merged commit 3db3a4c into flutter:master May 19, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 19, 2021
fluttergithubbot pushed a commit to flutter/flutter that referenced this pull request May 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes platform-web Code specifically for the web engine waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.
Projects
None yet
3 participants