Bug 190150

Summary: [Win][DRT] Actually set "experimental:WebAnimationsCSSIntegrationEnabled" when requested
Product: WebKit Reporter: Ross Kirsling <ross.kirsling>
Component: New BugsAssignee: Ross Kirsling <ross.kirsling>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, dino, graouts, Hironori.Fujii, pvollan, tsavell, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 190032    
Attachments:
Description Flags
Patch none

Description Ross Kirsling 2018-10-01 11:40:48 PDT
[Win][DRT] Actually set "experimental:WebAnimationsCSSIntegrationEnabled" when requested
Comment 1 Ross Kirsling 2018-10-01 11:42:03 PDT
Created attachment 351280 [details]
Patch
Comment 2 Fujii Hironori 2018-10-01 18:04:52 PDT
Mac port DRT has more options. Don't you need them?
https://github.com/WebKit/webkit/blob/234976b18c6973855fa27c50eea2070878ea1488/Tools/DumpRenderTree/mac/DumpRenderTree.mm#L998
Comment 3 Ross Kirsling 2018-10-01 19:38:44 PDT
(In reply to Fujii Hironori from comment #2)
> Mac port DRT has more options. Don't you need them?
> https://github.com/WebKit/webkit/blob/
> 234976b18c6973855fa27c50eea2070878ea1488/Tools/DumpRenderTree/mac/
> DumpRenderTree.mm#L998

Hmm, it would certainly seem like it, but evidently none of the other options currently have setters available in IWebPreferencesPrivate.idl:
https://github.com/WebKit/webkit/blob/master/Source/WebKitLegacy/win/Interfaces/IWebPreferencesPrivate.idl

So I suppose it'd be better to add them in the future, if we ever end up needing them in WKL.
Comment 4 WebKit Commit Bot 2018-10-01 22:24:51 PDT
Comment on attachment 351280 [details]
Patch

Clearing flags on attachment: 351280

Committed r236712: <https://trac.webkit.org/changeset/236712>
Comment 5 WebKit Commit Bot 2018-10-01 22:24:53 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2018-10-01 22:25:25 PDT
<rdar://problem/44931046>