RESOLVED FIXED176312
[WebIDL] Unify and simplify EnableBySettings with the rest of the runtime settings 
https://bugs.webkit.org/show_bug.cgi?id=176312
Summary [WebIDL] Unify and simplify EnableBySettings with the rest of the runtime set...
Sam Weinig
Reported 2017-09-03 09:28:30 PDT
[WebIDL] Simplify EnabledBySettings generation
Attachments
Patch (86.60 KB, patch)
2017-09-03 15:30 PDT, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Patch (89.06 KB, patch)
2017-09-03 16:32 PDT, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Patch (88.05 KB, patch)
2017-09-03 19:39 PDT, Sam Weinig 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2017-09-03 15:30:55 PDT
Created attachment 319810 [details] Patch
Sam Weinig
Comment 2 2017-09-03 15:37:10 PDT
Not thrilled with all the PublicName() suffixes, but I figure I should address that in a follow up.
Sam Weinig
Comment 3 2017-09-03 16:32:11 PDT
Created attachment 319815 [details] Patch
Sam Weinig
Comment 4 2017-09-03 19:39:54 PDT
Created attachment 319828 [details] Patch
WebKit Commit Bot
Comment 5 2017-09-04 03:51:02 PDT
Comment on attachment 319828 [details] Patch Clearing flags on attachment: 319828 Committed r221585: <http://trac.webkit.org/changeset/221585>
WebKit Commit Bot
Comment 6 2017-09-04 03:51:04 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 7 2017-09-27 12:39:27 PDT
<rdar://problem/34693695>
Note You need to log in before you can comment on or make changes to this bug.