RESOLVED FIXED 225041
Add an option to not enable all experimental features in DumpRenderTree
https://bugs.webkit.org/show_bug.cgi?id=225041
Summary Add an option to not enable all experimental features in DumpRenderTree
Ryosuke Niwa
Reported 2021-04-25 19:11:16 PDT
We need an option to avoid enabling all experimental features in DumpRenderTree so that we can run tests with the set of features we're intending to ship / are stable.
Attachments
Adds a flag (2.75 KB, patch)
2021-04-25 19:13 PDT, Ryosuke Niwa
no flags
Ryosuke Niwa
Comment 1 2021-04-25 19:13:12 PDT
Created attachment 427018 [details] Adds a flag
Wenson Hsieh
Comment 2 2021-04-25 19:15:35 PDT
Comment on attachment 427018 [details] Adds a flag View in context: https://bugs.webkit.org/attachment.cgi?id=427018&action=review r=mews > Tools/DumpRenderTree/mac/DumpRenderTree.mm:273 > +static int enableAllExperimentalFeatures = YES; Nit - BOOL?
Ryosuke Niwa
Comment 3 2021-04-25 19:54:42 PDT
Comment on attachment 427018 [details] Adds a flag View in context: https://bugs.webkit.org/attachment.cgi?id=427018&action=review >> Tools/DumpRenderTree/mac/DumpRenderTree.mm:273 >> +static int enableAllExperimentalFeatures = YES; > > Nit - BOOL? I think getopt_long wants these to be int's, not bool's.
Ryosuke Niwa
Comment 4 2021-04-25 22:51:26 PDT
Comment on attachment 427018 [details] Adds a flag Clearing flags on attachment: 427018 Committed r276581 (237017@main): <https://commits.webkit.org/237017@main>
Ryosuke Niwa
Comment 5 2021-04-25 22:51:28 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2021-04-25 22:52:19 PDT
Note You need to log in before you can comment on or make changes to this bug.