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

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Dec 20, 2019

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

Copy link
Contributor Author

@yjbanov yjbanov Dec 20, 2019

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
Copy link
Contributor

flar commented Dec 20, 2019

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

@yjbanov yjbanov merged commit 3381392 into flutter: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/engine#14590)

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

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

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

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

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

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

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

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

* 3381392 [web] implement pushImageFilter (flutter/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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants