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

Fix filter null exceptions#36575

Merged
auto-submit[bot] merged 7 commits intoflutter:mainfrom
alanwutang11:fix-filter-null-exceptions
Oct 4, 2022
Merged

Fix filter null exceptions#36575
auto-submit[bot] merged 7 commits intoflutter:mainfrom
alanwutang11:fix-filter-null-exceptions

Conversation

@alanwutang11
Copy link
Contributor

two issues fixed for canvaskit:

  1. ColorFilter.mode with a color opacity of zero would throw
  2. ImageFilter.blur with sigmaX and sigmaY of zero would throw

fixed by returning the identity matrix in both cases. Mimics NOP behavior, where the filter does nothing.

This is a temporary fix. Will create a better performing fix to handle the above listed edge cases so that we don't have to do the extra work of returning the identity matrix

see flutter/flutter#89433

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 the platform-web Code specifically for the web engine label Oct 3, 2022
@skia-gold
Copy link

Gold has detected about 1 new digest(s) on patchset 5.
View them at https://flutter-engine-gold.skia.org/cl/github/36575

@flutter-dashboard
Copy link

Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change).

If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review.

Changes reported for pull request #36575 at sha d280c59

@alanwutang11 alanwutang11 added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 4, 2022
@auto-submit
Copy link
Contributor

auto-submit bot commented Oct 4, 2022

auto label is removed for flutter/engine, pr: 36575, 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 auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 4, 2022
@auto-submit
Copy link
Contributor

auto-submit bot commented Oct 4, 2022

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

@skia-gold
Copy link

Gold has detected about 1 new digest(s) on patchset 6.
View them at https://flutter-engine-gold.skia.org/cl/github/36575

@flutter-dashboard
Copy link

Golden file changes are available for triage from new commit, Click here to view.

Changes reported for pull request #36575 at sha 5ff3c92

Copy link
Contributor

@harryterkelsen harryterkelsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with nit

@alanwutang11 alanwutang11 added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 4, 2022
@skia-gold
Copy link

Gold has detected about 1 new digest(s) on patchset 7.
View them at https://flutter-engine-gold.skia.org/cl/github/36575

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine will affect goldens

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants