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

[Impeller] Move GetSecondsElapsed to Playground #38375

Merged
merged 1 commit into from
Dec 17, 2022

Conversation

bdero
Copy link
Member

@bdero bdero commented Dec 17, 2022

PlaygroundTest isn't the right abstraction for time utils like this.

@bdero bdero self-assigned this Dec 17, 2022
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

Lgtm w/ nit

@@ -9,6 +9,7 @@
#include "flutter/fml/closure.h"
#include "flutter/fml/macros.h"

#include "fml/time/time_delta.h"
Copy link
Member

Choose a reason for hiding this comment

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

nit: Group with the fml includes above and prefix with "flutter/".

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@bdero bdero force-pushed the bdero/move-playground-time branch from 710e6c6 to 91c5068 Compare December 17, 2022 19:08
@bdero bdero added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 17, 2022
@auto-submit auto-submit bot merged commit 7e29698 into flutter:main Dec 17, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 17, 2022
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Dec 17, 2022
…117270)

* 1ae4c7c35 [Impeller Scene] Fix crasher for nodes with no meshes (flutter/engine#38374)

* 7e296985f [Impeller] Move GetSecondsElapsed to Playground (flutter/engine#38375)
loic-sharma pushed a commit to loic-sharma/flutter-engine that referenced this pull request Jan 3, 2023
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…lutter#117270)

* 1ae4c7c35 [Impeller Scene] Fix crasher for nodes with no meshes (flutter/engine#38374)

* 7e296985f [Impeller] Move GetSecondsElapsed to Playground (flutter/engine#38375)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App e: impeller
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants