Bug 240930 - Update internal CSS to remove '-webkit-' prefixing where possible
Summary: Update internal CSS to remove '-webkit-' prefixing where possible
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sammy Gill
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-05-25 16:15 PDT by Brent Fulgham
Modified: 2022-05-29 12:58 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2022-05-25 16:15:54 PDT
WebKit sources contain many instances of prefixed CSS properties. We should replace each with the equivalent unprefixed-property:

Source/WebCore/Modules/mediacontrols/mediaControlsAdwaita.css
192	  -webkit-user-select: none;
229	  -webkit-appearance: media-slider !important;
252	  -webkit-appearance: media-volume-slider-container;
285	  -webkit-appearance: media-volume-slider !important;
316	  -webkit-appearance: none !important;
351	  -webkit-appearance: media-closed-captions-container;
367	  -webkit-user-select: none;
476	  -webkit-flex: 1 1 auto;
493	  -webkit-user-select: none;

Source/WebCore/Modules/mediacontrols/mediaControlsApple.css
47	  -webkit-align-items: stretch;
48	  -webkit-justify-content: flex-end;
49	  -webkit-flex-direction: column;
62	  -webkit-flex: 1 1 auto;
74	  -webkit-user-select: none;
79	  -webkit-flex-direction: row;
80	  -webkit-align-items: center;
81	  -webkit-user-select: none;
123	  -webkit-appearance: none;
141	  -webkit-clip-path: inset(20px 0px 0px 0px);
180	  -webkit-appearance: none;
198	  -webkit-transform: translateZ(0);
241	  -webkit-flex-direction: column;
242	  -webkit-justify-content: center;
243	  -webkit-align-items: center;
262	  -webkit-clip-path: inset(20px 20px 20px 20px round 4px 4px 0px 0px);
267	  -webkit-flex-direction: row;
268	  -webkit-align-items: center;
269	  -webkit-justify-content: flex-end;
328	  -webkit-appearance: none !important;
339	  -webkit-transform-origin: 0 0;
340	  -webkit-transform: rotate(-90deg) translateY(28px) translateX(-40px);
349	  -webkit-appearance: none !important;
386	  -webkit-appearance: media-closed-captions-container;
406	  -webkit-user-select: none;
526	  -webkit-flex: 1 1 0;
537	  -webkit-appearance: none !important;
538	  -webkit-flex: 1 1 0 !important;
552	  -webkit-appearance: none !important;
561	  -webkit-user-select: none;
562	  -webkit-flex: 0 0 0;
595	  -webkit-justify-content: flex-end;
604	  -webkit-justify-content: flex-start;
630	  -webkit-flex-direction: row;
631	  -webkit-align-items: center;
632	  -webkit-user-select: none;
633	  -webkit-flex: 1 1 0;
642	  -webkit-flex: 1 1 0;
647	  -webkit-align-items: stretch;
648	  -webkit-flex-direction: column;
698	  -webkit-align-items: flex-start !important;
699	  -webkit-justify-content: flex-end !important;
700	  -webkit-clip-path: inset(20px round 6px);
744	  -webkit-transform: none;
745	  -webkit-clip-path: none;
752	  -webkit-flex-direction: row;
753	  -webkit-align-items: center;
763	  -webkit-transform: none;
898	  -webkit-user-select: none;
900	  -webkit-flex: 1 1;
913	  -webkit-box-sizing: border-box;
1052	  -webkit-user-select: none;

Source/WebCore/Modules/mediacontrols/mediaControlsBase.css
56	  -webkit-align-items: stretch;
57	  -webkit-justify-content: flex-end;
58	  -webkit-flex-direction: column;
64	  -webkit-flex: 1 1 auto;
68	  -webkit-transform: translateZ(0);
82	  -webkit-user-select: none;
98	  -webkit-flex-direction: row;
99	  -webkit-align-items: center;
100	  -webkit-user-select: none;
121	  -webkit-appearance: none;
132	  -webkit-filter: drop-shadow(black 0 1px 1px);
137	  -webkit-filter: drop-shadow(white 0 0 10px);
170	  -webkit-flex-direction: column;
171	  -webkit-justify-content: center;
172	  -webkit-align-items: center;
190	  -webkit-transform: rotate(-90deg);
191	  -webkit-transform-origin: 11px 11px;
203	  -webkit-flex-direction: row;
204	  -webkit-align-items: center;
205	  -webkit-justify-content: flex-end;
227	  -webkit-appearance: none !important;
247	  -webkit-appearance: none !important;
256	  -webkit-transform: rotateZ(0) !important;
292	  -webkit-appearance: media-closed-captions-container;
314	  -webkit-user-select: none;
414	  -webkit-flex: 1 1 0;
418	  -webkit-appearance: none !important;
419	  -webkit-flex: 1 1 0 !important;
429	  -webkit-appearance: none !important;
437	  -webkit-transform: translateY(1px) rotateZ(-45deg) !important;
464	  -webkit-user-select: none;
465	  -webkit-flex: 0 0 0;
467	  -webkit-justify-content: center;
468	  -webkit-align-items: center;
494	  -webkit-flex-direction: row;
495	  -webkit-align-items: center;
496	  -webkit-user-select: none;
497	  -webkit-flex: 1 1 0;
505	  -webkit-flex: 1 1 0;
509	  -webkit-align-items: stretch;
510	  -webkit-flex-direction: column;
553	  -webkit-align-items: flex-start !important;
554	  -webkit-justify-content: flex-end !important;
586	  -webkit-transform: none;
593	  -webkit-flex-direction: row;
594	  -webkit-align-items: center;
617	  -webkit-transform: rotateZ(270deg) !important;
716	  -webkit-user-select: none;
718	  -webkit-flex: 1 1;
731	  -webkit-box-sizing: border-box;

Source/WebCore/css/html.css
730	  -webkit-line-break: after-white-space;

Source/WebCore/css/legacyFormControlsIOS.css
69	  -webkit-mask-image: initial;

Source/WebCore/css/mediaControls.css
100	  -webkit-appearance: media-slider;

Source/WebCore/css/themeAdwaita.css
25	  -webkit-appearance: none;
26	  -webkit-box-shadow: none !important;
29	  -webkit-mask-image: url("data:image/svg+xml;utf-8, \
36	  -webkit-mask-size: 1em 1em;
40	  -webkit-appearance: none;
41	  -webkit-box-shadow: none !important;
44	  -webkit-mask-image: url("data:image/svg+xml;utf-8, \
48	  -webkit-mask-size: 1em 1em;
52	  -webkit-appearance: none;
53	  -webkit-box-shadow: none !important;
54	  -webkit-mask-image: url("data:image/svg+xml;utf-8, \
58	  -webkit-mask-size: 1em 1em;

Source/WebCore/html/shadow/imageOverlay.css
88	  -webkit-user-select: none;

Source/WebCore/html/shadow/mac/imageControlsMac.css
33	  -webkit-user-select: none;

Source/WebCore/html/shadow/meterElementShadow.css
4	  -webkit-appearance: inherit;

Source/WebCore/xml/XMLViewer.css
77	  -webkit-user-select: none;

Source/WebInspectorUI/UserInterface/External/CodeMirror/codemirror.css
65	  -webkit-animation: blink 1.06s steps(1) infinite;
227	  -webkit-user-select: none;
252	  -webkit-font-variant-ligatures: contextual;



Requested Changes:
===================
-webkit-align-items -> align-items
-webkit-animation -> animation
-webkit-appearance -> appearance
-webkit-box-shadow -> box-shadow
-webkit-box-sizing -> box-sizing
-webkit-clip-path -> clip-path
-webkit-flex -> flex
-webkit-flex-direction -> flex-direction
-webkit-filter -> filter
-webkit-justify-content -> justify-content
-webkit-line-break -> line-break
-webkit-mask-image -> mask-image
-webkit-mask-size -> mask-size
-webkit-transform -> transform
-webkit-user-select -> user-select
Comment 1 Radar WebKit Bug Importer 2022-05-25 16:16:12 PDT
<rdar://problem/93934318>
Comment 2 Tim Nguyen (:ntim) 2022-05-26 01:32:10 PDT
We'd probably want to avoid changing Source/WebInspectorUI/UserInterface/External/CodeMirror/codemirror.css which is imported.
Comment 3 Brent Fulgham 2022-05-26 11:04:35 PDT
(In reply to Tim Nguyen (:ntim) from comment #2)
> We'd probably want to avoid changing
> Source/WebInspectorUI/UserInterface/External/CodeMirror/codemirror.css which
> is imported.

Good point. I filed https://github.com/codemirror/CodeMirror/issues/6921 requesting they remove that deprecated markup from their trunk software.
Comment 4 Sammy Gill 2022-05-27 09:44:05 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1100
Comment 5 EWS 2022-05-27 16:02:48 PDT
Committed r294963 (251067@main): <https://commits.webkit.org/251067@main>

Reviewed commits have been landed. Closing PR #1100 and removing active labels.
Comment 6 Tim Nguyen (:ntim) 2022-05-29 12:50:01 PDT
Re-opening for pull request https://github.com/WebKit/WebKit/pull/1148
Comment 7 EWS 2022-05-29 12:57:12 PDT
Test gardening commit r295019 (251114@main): <https://commits.webkit.org/251114@main>

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