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

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Apr 21, 2022

Allows us to fix flutter/flutter#101866

Does the following:

  • Moves DartTimestampProvider to //flutter/runtime.
  • Installs that provider as the one to use for fml::TimePoint::Now when we initialize the Dart VM, falls back to stdlib's monotonic clock otherwise.
  • Removes direct calls to Dart_TimelineGetMicros and instead installs that as a callback when we initialize the Dart VM.

Verified locally that this allows successful builds of impeller without dependencies on the Dart VM.

@dnfield dnfield 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 Apr 21, 2022
@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Linux Fuchsia has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Linux Fuchsia FEMU has failed. Please fix the issues identified (or deflake) before re-applying this label.

@fluttergithubbot fluttergithubbot removed the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 21, 2022
@dnfield dnfield merged commit d415876 into flutter:main Apr 22, 2022
@dnfield dnfield deleted the fml_no_dart branch April 22, 2022 02:22
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

impellerc should not link in Dart

3 participants