WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 234024
[GPU Process] [Filters] Add the encoding and decoding for SVGFilter
https://bugs.webkit.org/show_bug.cgi?id=234024
Summary
[GPU Process] [Filters] Add the encoding and decoding for SVGFilter
Said Abou-Hallawa
Reported
2021-12-08 11:59:24 PST
This will allow SVGFilter and referenced SVGFilter to be applied in GPUProcess.
Attachments
Patch
(18.12 KB, patch)
2021-12-08 12:17 PST
,
Said Abou-Hallawa
wenson_hsieh
: review+
ews-feeder
: commit-queue-
Details
Formatted Diff
Diff
Patch
(18.19 KB, patch)
2021-12-08 12:39 PST
,
Said Abou-Hallawa
ews-feeder
: commit-queue-
Details
Formatted Diff
Diff
Patch
(18.77 KB, patch)
2021-12-08 14:37 PST
,
Said Abou-Hallawa
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Said Abou-Hallawa
Comment 1
2021-12-08 12:17:55 PST
Created
attachment 446409
[details]
Patch
Wenson Hsieh
Comment 2
2021-12-08 12:28:15 PST
Comment on
attachment 446409
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=446409&action=review
> Source/WebKit/Platform/IPC/FilterReference.h:334 > + auto expressionReference = WTF::map(filter.expression(), [&indecies] (auto&& term) -> ExpressionReferenceTerm {
Nit - sp. indecies => indicies
> Source/WebKit/Platform/IPC/FilterReference.h:383 > + WebCore::SVGFilterExpression expression;
Nit - maybe reserveInitialCapacity here?
Said Abou-Hallawa
Comment 3
2021-12-08 12:39:31 PST
Created
attachment 446410
[details]
Patch
Said Abou-Hallawa
Comment 4
2021-12-08 14:37:46 PST
Created
attachment 446422
[details]
Patch
EWS
Comment 5
2021-12-08 16:37:40 PST
Committed
r286753
(
244996@main
): <
https://commits.webkit.org/244996@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 446422
[details]
.
Radar WebKit Bug Importer
Comment 6
2021-12-08 16:38:25 PST
<
rdar://problem/86240715
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug