Bug 228697

Summary: Replace webkit- prefix properties with Standard Properties in Source/
Product: WebKit Reporter: Sonia Singla <soniasingla.1812>
Component: CSSAssignee: Sonia Singla <soniasingla.1812>
Status: RESOLVED FIXED    
Severity: Normal CC: calvaris, dbarton, eric.carlson, esprehn+autocc, ews-watchlist, fred.wang, glenn, gsnedders, gyuyoung.kim, jer.noble, macpherson, menard, obrufau, philipj, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 228546    
Attachments:
Description Flags
Patch
none
Patch none

Description Sonia Singla 2021-08-02 03:17:48 PDT
Replace webkit- prefix properties with Standard Properties in Source/ directory
Comment 1 Sonia Singla 2021-08-02 05:14:17 PDT
Created attachment 434747 [details]
Patch
Comment 2 Sam Sneddon [:gsnedders] 2021-08-02 06:24:15 PDT
Comment on attachment 434747 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=434747&action=review

> Source/WebCore/Modules/mediacontrols/mediaControlsApple.css:414
> -    -webkit-margin-start: 16px;
> +    margin-block-start: 16px;

You've consistently changed -webkit-margin-start to margin-block-start, but it should be margin-inline-start. I suspect this accounts for all the failures on EWS.
Comment 3 Sonia Singla 2021-08-02 07:46:16 PDT
Created attachment 434750 [details]
Patch
Comment 4 Sonia Singla 2021-08-02 20:02:46 PDT
>  You've consistently changed -webkit-margin-start to margin-block-start, but it should be margin-inline-start. I suspect this accounts for all the failures on EWS.

Done, sorry it was done by mistake with find and replace
Comment 5 EWS 2021-08-02 20:37:02 PDT
Committed r280581 (240203@main): <https://commits.webkit.org/240203@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 434750 [details].
Comment 6 Radar WebKit Bug Importer 2021-08-02 20:38:19 PDT
<rdar://problem/81442373>