This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Use non-deprecated SkImageFilter factory functions #23649
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
SkImageFilter::MakeMatrixFilter -> SkImageFilters::MatrixTransform
jason-simmons
approved these changes
Jan 13, 2021
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
cbracken
pushed a commit
to flutter/flutter
that referenced
this pull request
Jan 14, 2021
* d7b25f9 [web] Reland: Enable the new rich paragraph implementation (flutter/engine#23659) * 507a887 fix ax unique id flake (flutter/engine#23518) * e5a4c7c Roll Fuchsia Linux SDK from BUsKF6z4t... to a7ezEWPM5... (flutter/engine#23668) * 7d1e4e3 Roll Skia from 0e4a29af9db2 to af9b58e287b5 (10 revisions) (flutter/engine#23670) * 635f64a Roll Skia from af9b58e287b5 to f435ada424df (1 revision) (flutter/engine#23671) * 6957611 Roll Fuchsia Mac SDK from BsUY1yjWh... to BoBy7Eobf... (flutter/engine#23672) * 37131f1 Use non-deprecated SkImageFilter factory functions (flutter/engine#23649) * 38f08ff Roll Fuchsia Linux SDK from a7ezEWPM5... to erwxDS3kf... (flutter/engine#23676) * a56183c Roll Skia from f435ada424df to 8f78d5528438 (7 revisions) (flutter/engine#23680) * 9d75996 Revert "[web] Enable the new rich paragraph implementation (#23162)" (flutter/engine#23687) * b030bc8 Roll Dart SDK from 724d9e5e7d71 to 3629a798353d (1 revision) (flutter/engine#23681) * a8a7534 Support Mice in iPadOS 13.4+ (flutter/engine#23636) * 3f691f0 [web] Fix semantic node order for webkit (flutter/engine#23601) * 70d08c6 Roll Skia from 8f78d5528438 to 890b2b406a60 (9 revisions) (flutter/engine#23692) * 5459565 Roll Fuchsia Mac SDK from BoBy7Eobf... to DVYrV15dq... (flutter/engine#23693) * 3fd4c63 [web] Fix letter spacing for rich paragraphs (flutter/engine#23683) * effb529 Added missing export for the flutter engine group. (flutter/engine#23689)
gspencergoog
pushed a commit
to gspencergoog/engine
that referenced
this pull request
Jan 20, 2021
* Only use factory functions from SkImageFilters.h Fully addresses flutter/flutter#40602 * Update image filter factories in unittests SkImageFilter::MakeMatrixFilter -> SkImageFilters::MatrixTransform * Fix header
hjfreyer
pushed a commit
to hjfreyer/engine
that referenced
this pull request
Mar 22, 2021
* Only use factory functions from SkImageFilters.h Fully addresses flutter/flutter#40602 * Update image filter factories in unittests SkImageFilter::MakeMatrixFilter -> SkImageFilters::MatrixTransform * Fix header
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Updates SkPictureImageFilter::Make -> SkImageFilters::Picture and SkImageSource::Make -> SkImageFilters::Image. Also deletes unused image filter related headers and fixes unit tests that were making SkImageFilters directly.
Will address https://bugs.chromium.org/p/skia/issues/detail?id=9310 given that other image filter factories in image_filter.cc have already been updated to use SkImageFilters.h
Related Issues
flutter/flutter#40602 (will be fixed after this)
https://bugs.chromium.org/p/skia/issues/detail?id=9310 (this PR fixes the Flutter-side changes necessary to close the Skia issue).
Tests
No behavioral changes, so no new tests needed.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]). This will ensure a smooth and quick review process.Reviewer Checklist
Breaking Change
Did any tests fail when you ran them? Please read handling breaking changes.