-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Implement pushColorFilter in CanvasKit #22838
Implement pushColorFilter in CanvasKit #22838
Conversation
I could also wait for the SkPicture resurrection PR to land and add a golden test for this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -49,6 +49,17 @@ void testMain() { | |||
final ui.Image sceneImage = await scene.toImage(100, 100); | |||
expect(sceneImage, isA<CkImage>()); | |||
}); | |||
// TODO: https://github.com/flutter/flutter/issues/60040 | |||
|
|||
test('pushColorFilter does not throw', () async { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bonus points for making it a screenshot test. To do that, simply rename the file to scene_golden_test.dart
and use golden_tester.dart like normal (example).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
although the test would need to be skipped outside chrome.
Can we land this? |
Description
Implements
pushColorFilter
inui.SceneBuilder
in CanvasKit.Related Issues
Fixes the following issues in CanvasKit mode:
Fixes flutter/flutter#71733
Tests
I added the following tests:
Added a test to
scene_test.dart
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.