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

Fix includes of SkColorSpace et al#32382

Merged
jason-simmons merged 1 commit intoflutter:mainfrom
kjlubick:skcolorspace
Apr 1, 2022
Merged

Fix includes of SkColorSpace et al#32382
jason-simmons merged 1 commit intoflutter:mainfrom
kjlubick:skcolorspace

Conversation

@kjlubick
Copy link
Contributor

@kjlubick kjlubick commented Apr 1, 2022

The Skia team is cleaning up our headers. This PR is required to land https://skia-review.googlesource.com/c/skia/+/526416

See issue https://bugs.chromium.org/p/skia/issues/detail?id=13052

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.

@flutter-dashboard flutter-dashboard bot added embedder Related to the embedder API platform-android labels Apr 1, 2022
@kjlubick
Copy link
Contributor Author

kjlubick commented Apr 1, 2022

:shipit:

@jason-simmons jason-simmons 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 1, 2022
@fluttergithubbot
Copy link
Contributor

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

  • 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.

@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 1, 2022
@jason-simmons jason-simmons merged commit 5f2b73e into flutter:main Apr 1, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 1, 2022
chri7325 pushed a commit to Esri/skia that referenced this pull request May 2, 2022
…fo""

This is a reland of commit 00de8cc

New client fixes:
 - http://cl/445903108

Original change's description:
> Reland "[includes] Remove SkColorSpace include from SkImageInfo"
>
> This is a reland of commit 2151fa3
>
> New client fixes:
>  - https://crrev.com/c/3616345
>
> Original change's description:
> > [includes] Remove SkColorSpace include from SkImageInfo
> >
> > See also https://skia-review.googlesource.com/c/skia/+/525639
> >
> > This should be landed when clients have been fixed.
> >  - flutter/engine#32532
> >  - flutter/engine#32382
> >  - http://cl/438808657
> >  - http://cl/438803822
> >  - http://cl/438807456
> >  - http://cl/438808026
> >  - http://cl/438821192
> >  - https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/17542295
> >
> > Change-Id: I04beeb2e2b6132c40508c98f4f281981dadbb0d4
> > Bug: skia:13052
> > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/526416
> > Commit-Queue: Kevin Lubick <[email protected]>
> > Reviewed-by: Brian Salomon <[email protected]>
>
> Canary-Chromium-CL: 3616345
> Bug: skia:13052
> Change-Id: I670d3d0d0bab2a86559d9a1184cb36ef008196f3
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/535357
> Reviewed-by: Brian Salomon <[email protected]>

Bug: skia:13052
Change-Id: Ibb931dde3f0846609df35d2897989eff0499b7f9
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/535896
Commit-Queue: Kevin Lubick <[email protected]>
Reviewed-by: Brian Salomon <[email protected]>
@kjlubick kjlubick deleted the skcolorspace branch August 3, 2023 18:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

embedder Related to the embedder API platform-android

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants