Bug 240930

Summary: Update internal CSS to remove '-webkit-' prefixing where possible
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebKit Misc.Assignee: Sammy Gill <sgill26>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, ntim, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Brent Fulgham
Reported 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
Attachments
Radar WebKit Bug Importer
Comment 1 2022-05-25 16:16:12 PDT
Tim Nguyen (:ntim)
Comment 2 2022-05-26 01:32:10 PDT
We'd probably want to avoid changing Source/WebInspectorUI/UserInterface/External/CodeMirror/codemirror.css which is imported.
Brent Fulgham
Comment 3 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.
Sammy Gill
Comment 4 2022-05-27 09:44:05 PDT
EWS
Comment 5 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.
Tim Nguyen (:ntim)
Comment 6 2022-05-29 12:50:01 PDT
Re-opening for pull request https://github.com/WebKit/WebKit/pull/1148
EWS
Comment 7 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.
Note You need to log in before you can comment on or make changes to this bug.