RESOLVED FIXED Bug 65747
Remove LegacyDefaultOptionalArguments flag from notifications IDL files 
https://bugs.webkit.org/show_bug.cgi?id=65747
Summary Remove LegacyDefaultOptionalArguments flag from notifications IDL files
Mark Pilgrim (Google)
Reported 2011-08-04 22:16:50 PDT
As discussed in IRC, we are migrating our IDL files away from the interface-level "LegacyDefaultOptionalArguments" flag and onto argument-level [Optional] or [Optional=CallWithDefaultValue] flags. This patch migrates the notifications-related IDL files. It does not change any behavior, i.e. it does not make any arguments required that were previously optional, nor vice-versa. All existing tests pass.
Attachments
Patch (1.69 KB, patch)
2011-08-04 22:17 PDT, Mark Pilgrim (Google) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Pilgrim (Google)
Comment 1 2011-08-04 22:17:25 PDT
Created attachment 103036 [details] Patch
Adam Barth
Comment 2 2011-08-04 23:13:55 PDT
Comment on attachment 103036 [details] Patch Any sense for how far along we are?
WebKit Review Bot
Comment 3 2011-08-05 00:18:36 PDT
Comment on attachment 103036 [details] Patch Clearing flags on attachment: 103036 Committed r92454: <http://trac.webkit.org/changeset/92454>
WebKit Review Bot
Comment 4 2011-08-05 00:18:40 PDT
All reviewed patches have been landed. Closing bug.
Mark Pilgrim (Google)
Comment 5 2011-08-05 06:37:08 PDT
Only a handful left. Literally, less than 5.
Note You need to log in before you can comment on or make changes to this bug.