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

Windows: Make task_runner_unittests.cc more deterministic. #29500

Merged
merged 1 commit into from
Jan 19, 2022

Conversation

moko256
Copy link
Contributor

@moko256 moko256 commented Nov 3, 2021

This PR is cleanup #28013 from #29484 (comment).
This PR will enable task_runner_unittests.cc to mock TaskTimePoint::clock::now() by overriding, and make task_runner_unittests.cc more deterministic.

No tests in flutter/tests.

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See [testing the engine] for instructions on
    writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the [CLA].
  • All existing and new tests are passing.

@google-cla google-cla bot added the cla: yes label Nov 3, 2021
@CaseyHillers CaseyHillers changed the base branch from master to main November 15, 2021 18:13
@godofredoc godofredoc changed the base branch from master to main November 24, 2021 07:01
@flutter-dashboard
Copy link

This pull request executed golden file tests, but it has not been updated in a while (20+ days). Test results from Gold expire after as many days, so this pull request will need to be updated with a fresh commit in order to get results from Gold.

@bdero bdero merged commit 31b1a1d into flutter:main Jan 19, 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

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 19, 2022
zanderso pushed a commit to flutter/flutter that referenced this pull request Jan 19, 2022
* bdab647 Roll Skia from 9f231c330962 to 8276fa8ef102 (2 revisions) (flutter/engine#30912)

* 7002447 Remove redundant `--template` arguments from code samples (flutter/engine#30909)

* cef8bce [Web Text Input] ensure the input element is put in the DOM on desktop safari (flutter/engine#30885)

* 31b1a1d cleanup #28013 (flutter/engine#29500)
clocksmith pushed a commit to clocksmith/flutter that referenced this pull request Mar 8, 2022
* bdab647 Roll Skia from 9f231c330962 to 8276fa8ef102 (2 revisions) (flutter/engine#30912)

* 7002447 Remove redundant `--template` arguments from code samples (flutter/engine#30909)

* cef8bce [Web Text Input] ensure the input element is put in the DOM on desktop safari (flutter/engine#30885)

* 31b1a1d cleanup flutter#28013 (flutter/engine#29500)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants