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

[web] implement pushImageFilter#14599

Merged
yjbanov merged 3 commits into
flutter-team-archive:masterfrom
yjbanov:push-image-filter
Dec 20, 2019
Merged

[web] implement pushImageFilter#14599
yjbanov merged 3 commits into
flutter-team-archive:masterfrom
yjbanov:push-image-filter

Conversation

@yjbanov

@yjbanov yjbanov commented Dec 20, 2019

Copy link
Copy Markdown
Contributor

This is a follow-up to #14491. It implements the image filter layer for the web.

@auto-assign auto-assign Bot requested a review from gaaclarke December 20, 2019 16:49
@yjbanov yjbanov requested review from ferhatb and flar December 20, 2019 16:49

html.document.body.append(builder.build().webOnlyRootElement);

await matchGoldenFile('compositing_image_filter.png', region: region, write: true);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please remove write:true

@yjbanov yjbanov Dec 20, 2019

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@dnfield dnfield added the CQ+1 label Dec 20, 2019
@flar

flar commented Dec 20, 2019

Copy link
Copy Markdown
Contributor

I am not familiar with the details, but the parts that I understood LGTM...

@yjbanov yjbanov merged commit 3381392 into flutter-team-archive:master Dec 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 20, 2019
flar pushed a commit to flutter/flutter that referenced this pull request Dec 21, 2019
* 399a29e Roll src/third_party/dart 141fcfa61092..4681868aaf77 (9 commits) (flutter-team-archive/engine#14590)

* 97fd705 Roll src/third_party/skia 7e2dea568299..dcac29b02d02 (3 commits) (flutter-team-archive/engine#14591)

* 6d92887 Roll src/third_party/dart 4681868aaf77..b92fefdfe971 (1 commits) (flutter-team-archive/engine#14592)

* 4d7313f Roll fuchsia/sdk/core/mac-amd64 from f51Q_... to VC7eE... (flutter-team-archive/engine#14593)

* fac66c2 [web] Long press fix on Safari on IOS (flutter-team-archive/engine#14588)

* 32eb6e9 Roll src/third_party/skia dcac29b02d02..088913a63b7e (1 commits) (flutter-team-archive/engine#14597)

* ede1b6b Roll fuchsia/sdk/core/linux-amd64 from 25LzW... to TUoVa... (flutter-team-archive/engine#14596)

* cce5afd Roll src/third_party/dart b92fefdfe971..886615d0f95b (2 commits) (flutter-team-archive/engine#14595)

* 9b299f2 fix editable placement in a11y mode (flutter-team-archive/engine#14581)

* 3381392 [web] implement pushImageFilter (flutter-team-archive/engine#14599)
filmil pushed a commit to filmil/engine that referenced this pull request Mar 13, 2020
implement pushImageFilter in the HTML and CanvasKit backends
@yjbanov yjbanov deleted the push-image-filter branch June 22, 2021 21:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

5 participants