RESOLVED FIXED 222868
Transition Cocoa DumpRenderTree to allow any preference keys, not just hard coded supported ones 
https://bugs.webkit.org/show_bug.cgi?id=222868
Summary Transition Cocoa DumpRenderTree to allow any preference keys, not just hard c...
Sam Weinig
Reported 2021-03-06 12:24:59 PST
Transition Cocoa DumpRenderTree to allow any preference key, not just hard codded supported one
Attachments
Patch (18.21 KB, patch)
2021-03-06 12:30 PST, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Patch (17.00 KB, patch)
2021-03-06 18:21 PST, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2021-03-06 12:30:48 PST Comment hidden (obsolete)
Sam Weinig
Comment 2 2021-03-06 18:21:55 PST
Created attachment 422515 [details] Patch
Simon Fraser (smfr)
Comment 3 2021-03-07 09:28:45 PST
Comment on attachment 422515 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=422515&action=review > Tools/DumpRenderTree/TestOptions.cpp:123 > + { "AspectRatioOfImgFromWidthAndHeightEnabled", false }, > + { "AsyncClipboardAPIEnabled", false }, > + { "CSSOMViewSmoothScrollingEnabled", false }, > + { "ContactPickerAPIEnabled", false }, > + { "CoreMathMLEnabled", false }, > + { "GenericCueAPIEnabled", false }, > + { "IntersectionObserverEnabled", false }, > + { "IsLoggedInAPIEnabled", false }, > + { "LazyIframeLoadingEnabled", false }, > + { "LazyImageLoadingEnabled", false }, > + { "RequestIdleCallbackEnabled", false }, > + { "ResizeObserverEnabled", false }, > + { "WebAuthenticationEnabled", false }, > + { "WebGPUEnabled", false }, Seems like at least some of these should just be enabled in WK1. Like, AspectRatioOfImgFromWidthAndHeightEnabled is purely about layout.
EWS
Comment 4 2021-03-07 09:41:55 PST
Committed r274052: <https://commits.webkit.org/r274052> All reviewed patches have been landed. Closing bug and clearing flags on attachment 422515 [details].
Radar WebKit Bug Importer
Comment 5 2021-03-07 09:42:17 PST
<rdar://problem/75148917>
Note You need to log in before you can comment on or make changes to this bug.