Skip to content
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

Relax the checks of the alpha channel of the blurred stroke in fast/canvas/canvas-scale-strokePath-shadow.html #9491

Conversation

shallawa
Copy link
Contributor

@shallawa shallawa commented Feb 2, 2023

d538ed7

Relax the checks of the alpha channel of the blurred stroke in fast/canvas/canvas-scale-strokePath-shadow.html
https://bugs.webkit.org/show_bug.cgi?id=251558
rdar://104761905

Reviewed by Dean Jackson.

Relax the check for the alpha channel of the blurred stroke in this test to avoid
rebase-line the expectation in the future. And git rid of the platform expectation.

Ensure the alpha channel of the blurred stroke is between 75 and 225 instead of
175 and 225.

* LayoutTests/fast/canvas/canvas-scale-strokePath-shadow-expected.txt:
* LayoutTests/fast/canvas/canvas-scale-strokePath-shadow.html:
* LayoutTests/platform/ios/fast/canvas/canvas-scale-strokePath-shadow-expected.txt: Removed.
* LayoutTests/platform/mac/fast/canvas/canvas-scale-strokePath-shadow-expected.txt: Removed.
* LayoutTests/platform/win/fast/canvas/canvas-scale-strokePath-shadow-expected.txt: Removed.

Canonical link: https://commits.webkit.org/259765@main

17eab40

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe βœ… πŸ›  πŸ§ͺ win
βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug βœ… πŸ›  gtk βœ… πŸ›  wincairo
βœ… πŸ§ͺ webkitperl βœ… πŸ§ͺ ios-wk2 βœ… πŸ§ͺ api-mac βœ… πŸ§ͺ gtk-wk2
βœ… πŸ§ͺ api-ios βœ… πŸ§ͺ mac-wk1 βœ… πŸ§ͺ api-gtk
βœ… πŸ›  tv βœ… πŸ§ͺ mac-wk2
βœ… πŸ›  tv-sim βœ… πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  watch βœ… πŸ§ͺ mac-wk2-stress
βœ… πŸ›  πŸ§ͺ merge βœ… πŸ›  watch-sim

@shallawa shallawa self-assigned this Feb 2, 2023
@shallawa shallawa added the Canvas Bugs related to the canvas element. label Feb 2, 2023
@shallawa shallawa requested a review from smfr February 2, 2023 00:03
@grorg grorg added the merge-queue Applied to send a pull request to merge-queue label Feb 2, 2023
…anvas/canvas-scale-strokePath-shadow.html

https://bugs.webkit.org/show_bug.cgi?id=251558
rdar://104761905

Reviewed by Dean Jackson.

Relax the check for the alpha channel of the blurred stroke in this test to avoid
rebase-line the expectation in the future. And git rid of the platform expectation.

Ensure the alpha channel of the blurred stroke is between 75 and 225 instead of
175 and 225.

* LayoutTests/fast/canvas/canvas-scale-strokePath-shadow-expected.txt:
* LayoutTests/fast/canvas/canvas-scale-strokePath-shadow.html:
* LayoutTests/platform/ios/fast/canvas/canvas-scale-strokePath-shadow-expected.txt: Removed.
* LayoutTests/platform/mac/fast/canvas/canvas-scale-strokePath-shadow-expected.txt: Removed.
* LayoutTests/platform/win/fast/canvas/canvas-scale-strokePath-shadow-expected.txt: Removed.

Canonical link: https://commits.webkit.org/259765@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/Relax-the-checks-of-the-alpha-channel-of-the-blurred-stroke-in-fastcanvascanvas-scale-strokePath-shadow-html branch from 17eab40 to d538ed7 Compare February 2, 2023 19:34
@webkit-commit-queue
Copy link
Collaborator

Committed 259765@main (d538ed7): https://commits.webkit.org/259765@main

Reviewed commits have been landed. Closing PR #9491 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit d538ed7 into WebKit:main Feb 2, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Feb 2, 2023
@shallawa shallawa deleted the eng/Relax-the-checks-of-the-alpha-channel-of-the-blurred-stroke-in-fastcanvascanvas-scale-strokePath-shadow-html branch May 9, 2023 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Canvas Bugs related to the canvas element.
Projects
None yet
4 participants