RESOLVED FIXED 234602
Internally unprefix -webkit-text-emphasis properties 
https://bugs.webkit.org/show_bug.cgi?id=234602
Summary Internally unprefix -webkit-text-emphasis properties
Tim Nguyen (:ntim)
Reported 2021-12-22 06:18:28 PST
It's already exposed unprefixed externally, let's reflect this internally as well. Makes it easier to remove prefixed versions later on if needed, and it's also less confusing when going through the CSSProperties.json file.
Attachments
Patch (18.07 KB, patch)
2021-12-22 07:01 PST, Tim Nguyen (:ntim) 		koivisto: review+
ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (32.66 KB, patch)
2021-12-22 08:55 PST, Tim Nguyen (:ntim) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tim Nguyen (:ntim)
Comment 1 2021-12-22 07:01:13 PST
Created attachment 447803 [details] Patch
Tim Nguyen (:ntim)
Comment 2 2021-12-22 08:55:18 PST
Created attachment 447806 [details] Patch
Tim Nguyen (:ntim)
Comment 3 2021-12-22 08:58:02 PST
Committed r287356 (245498@trunk): <https://commits.webkit.org/245498@trunk>
Radar WebKit Bug Importer
Comment 4 2021-12-22 08:58:17 PST
<rdar://problem/86813136>
Tim Nguyen (:ntim)
Comment 5 2021-12-23 09:56:12 PST
Test gardening for ios done in: https://commits.webkit.org/r287403
Darin Adler
Comment 6 2021-12-24 09:19:30 PST
Comment on attachment 447806 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=447806&action=review > Source/WebCore/style/StyleBuilderCustom.h:115 > - DECLARE_PROPERTY_CUSTOM_HANDLERS(WebkitTextEmphasisStyle); > + DECLARE_PROPERTY_CUSTOM_HANDLERS(TextEmphasisStyle); Should we sort this one?
Tim Nguyen (:ntim)
Comment 7 2021-12-24 12:20:52 PST
(In reply to Darin Adler from comment #6) > Comment on attachment 447806 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=447806&action=review > > > Source/WebCore/style/StyleBuilderCustom.h:115 > > - DECLARE_PROPERTY_CUSTOM_HANDLERS(WebkitTextEmphasisStyle); > > + DECLARE_PROPERTY_CUSTOM_HANDLERS(TextEmphasisStyle); > > Should we sort this one? Fixed in https://commits.webkit.org/r287430
Note You need to log in before you can comment on or make changes to this bug.