-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Send timings of the first frame without batching #9424
Conversation
For flutter/flutter#34867 Test added: ReportTimingsIsCalledImmediatelyAfterTheFirstFrame
// Create the surface needed by rasterizer | ||
PlatformViewNotifyCreated(shell.get()); | ||
|
||
auto configuration = RunConfiguration::InferFromSettings(settings); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How is this working? You just moved the settings into the CreateShell call? Either make it const
or don't move it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, it seems that I forgot to remove std::move
when I removed &&
in the last PR. By the fact that all tests have passed in these days, it seems to me that std::move
may just be a no-op here?
Anyway, std::move
removed.
flutter/engine@297cbd4...b188479 git log 297cbd4..b188479 --no-merges --oneline b188479 Remove variants of ParagraphBuilder::AddText that are not used within the engine (flutter/engine#9403) a184037 Send timings of the first frame without batching (flutter/engine#9424) 215096c Roll src/third_party/skia 91032f8cd7ea..d8f79a27b06b (16 commits) (flutter/engine#9457) 52ebf4c Made sure that the run_tests script returns the right error code. (flutter/engine#9456) The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff (amirha@google.com), and stop the roller if necessary.
flutter/engine@297cbd4...b188479 git log 297cbd4..b188479 --no-merges --oneline b188479 Remove variants of ParagraphBuilder::AddText that are not used within the engine (flutter/engine#9403) a184037 Send timings of the first frame without batching (flutter/engine#9424) 215096c Roll src/third_party/skia 91032f8cd7ea..d8f79a27b06b (16 commits) (flutter/engine#9457) 52ebf4c Made sure that the run_tests script returns the right error code. (flutter/engine#9456) The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff (amirha@google.com), and stop the roller if necessary.
For flutter/flutter#34867
Test added: