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

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants