Thanks to visit codestin.com
Credit goes to github.com

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

Add necessary #includes for SkCodecAnimation.h #37040

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

kjlubick
Copy link
Contributor

The Skia team is cleaning up the #includes inside our public headers. These changes keep Flutter Engine from depending on transitive dependencies that are being removed.

Pre-launch Checklist

  • [ x ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • [ x ] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • [ x ] 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.
  • [ x ] 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.
  • [ x ] I updated/added relevant documentation (doc comments with ///).
  • [ x ] I signed the [CLA].
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on [Discord].

@jason-simmons jason-simmons added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 26, 2022
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 26, 2022
@auto-submit
Copy link
Contributor

auto-submit bot commented Oct 26, 2022

auto label is removed for flutter/engine, pr: 37040, due to - Please get at least one approved review if you are already a member or two member reviews if you are not a member before re-applying this label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

@auto-submit
Copy link
Contributor

auto-submit bot commented Oct 26, 2022

auto label is removed for flutter/engine, pr: 37040, due to Validations Fail.

@jason-simmons jason-simmons merged commit 31d21cb into flutter:main Oct 26, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 26, 2022
@kjlubick kjlubick deleted the fix-includes-of-animated-codec branch November 1, 2022 12:47
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.

2 participants