REOPENED 185087
Refactor filter list checking code 
https://bugs.webkit.org/show_bug.cgi?id=185087
Summary Refactor filter list checking code
Simon Fraser (smfr)
Reported 2018-04-27 13:33:49 PDT
Refactor filter list checking code
Attachments
Patch (6.17 KB, patch)
2018-04-27 13:35 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Follow-up fix of possible build failure (1.07 KB, patch)
2018-09-07 12:05 PDT, Frédéric Wang (:fredw) 		no flags
DetailsFormatted DiffDiff
Another follow-up fix of potential build failure (1006 bytes, patch)
2018-09-11 05:42 PDT, Frédéric Wang (:fredw) 		no flags
DetailsFormatted DiffDiff
Patch (3.43 KB, patch)
2018-09-12 09:03 PDT, Frédéric Wang (:fredw) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2018-04-27 13:35:24 PDT
Created attachment 339018 [details] Patch
Simon Fraser (smfr)
Comment 2 2018-04-27 15:02:17 PDT
https://trac.webkit.org/changeset/231112/webkit
Radar WebKit Bug Importer
Comment 3 2018-04-27 15:03:36 PDT
<rdar://problem/39805938>
Frédéric Wang (:fredw)
Comment 4 2018-09-07 07:50:41 PDT
Comment on attachment 339018 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=339018&action=review > Source/WebCore/animation/KeyframeEffectReadOnly.h:153 > + bool checkForMatchingFilterFunctionLists(CSSPropertyID, const std::function<const FilterOperations& (const RenderStyle&)>&) const; @smfr: I think we should really forward-declare FilterOperations here.
Simon Fraser (smfr)
Comment 5 2018-09-07 11:23:37 PDT
Sure.
Frédéric Wang (:fredw)
Comment 6 2018-09-07 12:05:52 PDT
Created attachment 349173 [details] Follow-up fix of possible build failure (In reply to Simon Fraser (smfr) from comment #5) > Sure. For the record, I'm found this after getting a build failure with a patch for bug 188043 ; probably because of the unified build thing.
Frédéric Wang (:fredw)
Comment 7 2018-09-09 23:23:20 PDT
smfr: review ping?
Frédéric Wang (:fredw)
Comment 8 2018-09-10 05:01:06 PDT
Committed r235847: <https://trac.webkit.org/changeset/235847>
Frédéric Wang (:fredw)
Comment 9 2018-09-11 03:00:38 PDT
(In reply to Simon Fraser (smfr) from comment #2) > https://trac.webkit.org/changeset/231112/webkit Mmh, that commit adds one more change with respect to the patch in Source/WebCore/page/animation/KeyframeAnimation.h that requires similar fixup.
Frédéric Wang (:fredw)
Comment 10 2018-09-11 05:42:51 PDT
Created attachment 349392 [details] Another follow-up fix of potential build failure
Antonio Gomes
Comment 11 2018-09-11 05:46:22 PDT
Comment on attachment 349392 [details] Another follow-up fix of potential build failure rs=me
Frédéric Wang (:fredw)
Comment 12 2018-09-11 05:48:43 PDT
Committed r235894: <https://trac.webkit.org/changeset/235894>
Frédéric Wang (:fredw)
Comment 13 2018-09-12 09:03:01 PDT
Reopening to attach new patch.
Frédéric Wang (:fredw)
Comment 14 2018-09-12 09:03:04 PDT
Created attachment 349552 [details] Patch
Myles C. Maxfield
Comment 15 2018-09-12 09:57:12 PDT
Comment on attachment 349552 [details] Patch We often inline functions for performance, and fonts are used pretty often. Is this a performance regression?
Myles C. Maxfield
Comment 16 2018-09-12 09:58:22 PDT
r=me if this isn’t a performance regression
Frédéric Wang (:fredw)
Comment 17 2018-09-12 11:30:27 PDT
Comment on attachment 349552 [details] Patch Mmh, wrong bug sorry...
Note You need to log in before you can comment on or make changes to this bug.