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

[Impeller] implement mask blur for textures #51183

Merged
gaaclarke merged 12 commits into from
Mar 8, 2024
Merged

[Impeller] implement mask blur for textures #51183

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
35843e8
[Impeller] implement mask blur for textures
gaaclarke Mar 4, 2024
0ff9413
okay a bit closer, this seems to mimic skia's behavior
gaaclarke Mar 5, 2024
81b2a32
removed fudge
gaaclarke Mar 6, 2024
88c766b
lint
gaaclarke Mar 7, 2024
11da599
renamed test
gaaclarke Mar 7, 2024
abd12a4
added golden update
gaaclarke Mar 7, 2024
66c4d38
removed color filter
gaaclarke Mar 7, 2024
3233ee2
removed changes to impeller golden tests output
gaaclarke Mar 8, 2024
6868bd4
added removed goldens back
gaaclarke Mar 8, 2024
2277ce1
Merge branch 'main' into mask-blur-texture
gaaclarke Mar 8, 2024
22246b5
Merge branch 'main' into mask-blur-texture
gaaclarke Mar 8, 2024
5c33bde
fixed merge conflict
gaaclarke Mar 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
removed color filter
  • Loading branch information
@gaaclarke
gaaclarke committed Mar 8, 2024
commit 66c4d3840c91a5d9ea70f0d645f66c9adafec840
13 changes: 1 addition & 12 deletions impeller/aiks/canvas.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -737,20 +737,9 @@ void Canvas::DrawImageRect(const std::shared_ptr<Image>& image,
texture_contents->SetOpacity(paint.color.alpha);
texture_contents->SetDeferApplyingOpacity(paint.HasColorFilter());

bool needs_color_filter = paint.HasColorFilter();
if (needs_color_filter) {
auto color_filter = paint.GetColorFilter();
if (texture_contents->ApplyColorFilter(
color_filter->GetCPUColorFilterProc())) {
needs_color_filter = false;
}
}

std::shared_ptr<Contents> contents = texture_contents;
if (paint.mask_blur_descriptor.has_value()) {
contents = paint.mask_blur_descriptor->CreateMaskBlur(
texture_contents,
needs_color_filter ? paint.GetColorFilter() : nullptr);
contents = paint.mask_blur_descriptor->CreateMaskBlur(texture_contents);
}

Entity entity;
Expand Down
3 changes: 1 addition & 2 deletions impeller/aiks/paint.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,8 +123,7 @@ std::shared_ptr<Contents> Paint::WithColorFilter(
}

std::shared_ptr<FilterContents> Paint::MaskBlurDescriptor::CreateMaskBlur(
Copy link
Contributor

@jonahwilliams jonahwilliams Mar 7, 2024

Choose a reason for hiding this comment

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

I would defer to @bdero to review the mask blur implementation. Overall code looks reasonable to me.

std::shared_ptr<TextureContents> texture_contents,
const std::shared_ptr<ColorFilter>& color_filter) const {
std::shared_ptr<TextureContents> texture_contents) const {
Scalar expand_amount = GaussianBlurFilterContents::CalculateBlurRadius(
GaussianBlurFilterContents::ScaleSigma(sigma.sigma));
texture_contents->SetSourceRect(
Expand Down
3 changes: 1 addition & 2 deletions impeller/aiks/paint.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,7 @@ struct Paint {
const std::shared_ptr<ColorFilter>& color_filter) const;

std::shared_ptr<FilterContents> CreateMaskBlur(
std::shared_ptr<TextureContents> texture_contents,
const std::shared_ptr<ColorFilter>& color_filter) const;
std::shared_ptr<TextureContents> texture_contents) const;

std::shared_ptr<FilterContents> CreateMaskBlur(
const FilterInput::Ref& input,
Expand Down