-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Revert "Add CanvasFilter objects as possible 2D context filters" #7874
Conversation
It did not have implementer agreement after all. This reverts commit 5db6a65.
@mysteryDate could you work marking the tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A shame to lose all the editorial cleanups that came along with this, but a clean revert is probably simpler.
Thanks for the review @domenic! @mysteryDate @fserb it would be ideal if the test changes can be made this week or so, so the specification isn't incorrect for too long a period. @whatwg/canvas this may be of interest. |
I pushed a commit that lets us preserve the nice editorial updates. I also worked on a CL to rename the tests. However in the meantime @annevk has lost WHATWG participation status :). So @annevk, at your leisure (please enjoy your vacation!), please review my extra commit, and maybe sign up as an individual. |
The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771
The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771
The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771
The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771
The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3714402 Reviewed-by: Aaron Krajeski <aaronhk@chromium.org> Commit-Queue: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/main@{#1018614}
The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3714402 Reviewed-by: Aaron Krajeski <aaronhk@chromium.org> Commit-Queue: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/main@{#1018614}
…, a=testonly Automatic update from web-platform-tests Rename CanvasFilter WPTs to be tentative The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3714402 Reviewed-by: Aaron Krajeski <aaronhk@chromium.org> Commit-Queue: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/main@{#1018614} -- wpt-commits: 0c579dc895666e989c046b706955ca19d1244c7d wpt-pr: 34536
…, a=testonly Automatic update from web-platform-tests Rename CanvasFilter WPTs to be tentative The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3714402 Reviewed-by: Aaron Krajeski <aaronhk@chromium.org> Commit-Queue: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/main@{#1018614} -- wpt-commits: 0c579dc895666e989c046b706955ca19d1244c7d wpt-pr: 34536
I cannot review my own PR, but it looks okay to me. I have updated the IPR check as well. I wasn't sure if we still wanted to use the first commit for the commit message so I'll leave that for you to decide. |
The spec is being reverted per whatwg/html#7874 due to lack of multi-implementer interest. So, we need to rename the tests. Bug: 1169216 Change-Id: I059fa4c951f2076cffb562bf4a32d4e4284d3771 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3714402 Reviewed-by: Aaron Krajeski <aaronhk@chromium.org> Commit-Queue: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/main@{#1018614} NOKEYCHECK=True GitOrigin-RevId: 08ace1810e694323dd0aa354baaf8a89e58b9bfc
It did not have implementer agreement after all.
This reverts commit 5db6a65.
See comments starting at #5621 (comment).
CanvasFilter
tests to tentative(See WHATWG Working Mode: Changes for more details.)
/canvas.html ( diff )
/infrastructure.html ( diff )