RESOLVED FIXED 65337
Remove LegacyDefaultOptionalArguments flag from SVG IDL files
https://bugs.webkit.org/show_bug.cgi?id=65337
Summary Remove LegacyDefaultOptionalArguments flag from SVG IDL files
Mark Pilgrim (Google)
Reported 2011-07-28 12:27:39 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 all remaining SVG-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 (28.40 KB, patch)
2011-07-28 12:28 PDT, Mark Pilgrim (Google)
no flags
Mark Pilgrim (Google)
Comment 1 2011-07-28 12:28:35 PDT
Nikolas Zimmermann
Comment 2 2011-07-28 12:45:42 PDT
In principle, I'm fine with this patch, as it preserves the existing behaviour. Though wouldn't it be a good idea to enforce strictness? In the past I've converted our primitives SVGLength/SVGAngle to RequiresAllArguments=Raise, to be more strict.
Adam Barth
Comment 3 2011-07-28 12:49:39 PDT
(In reply to comment #2) > In principle, I'm fine with this patch, as it preserves the existing behaviour. > Though wouldn't it be a good idea to enforce strictness? In the past I've converted our primitives SVGLength/SVGAngle to RequiresAllArguments=Raise, to be more strict. The main question is whether that's going to be a compatibility problem. In general, I'd like to align with Firefox here as much as possible, but I don't have a sense for how large the compat risk is.
Nikolas Zimmermann
Comment 4 2011-07-28 12:59:05 PDT
(In reply to comment #3) > (In reply to comment #2) > > In principle, I'm fine with this patch, as it preserves the existing behaviour. > > Though wouldn't it be a good idea to enforce strictness? In the past I've converted our primitives SVGLength/SVGAngle to RequiresAllArguments=Raise, to be more strict. > > The main question is whether that's going to be a compatibility problem. In general, I'd like to align with Firefox here as much as possible, but I don't have a sense for how large the compat risk is. I highly doubt it, but we won't know unless we try :-) We've aligned with FF regarding SVGAngle/SVGLength/... all SVG DOM primitives raise exceptions when setting wrong/invalid values (this is in contrary to what SVG 1.1 Second Edition specs - and may change in future).
Mark Pilgrim (Google)
Comment 5 2011-07-28 13:05:49 PDT
Enforcing strictness is a fine idea but really orthogonal to this particular patch. This is a step on the road of allowing us to get rid of the LegacyDefaultOptionalArguments flag and simplify the IDL code generator. I would be happy to be involved in a discussion and/or investigation of whether we can make some arguments required, but that's easy enough to do after this lands. In fact, it will be easier, since you will be able to control it on an argument-by-argument basis.
Nikolas Zimmermann
Comment 6 2011-07-28 13:08:11 PDT
Comment on attachment 102286 [details] Patch That argument convinced me, r=me, given no regressions change in behaviour.
WebKit Review Bot
Comment 7 2011-08-02 14:37:05 PDT
Comment on attachment 102286 [details] Patch Clearing flags on attachment: 102286 Committed r92237: <http://trac.webkit.org/changeset/92237>
WebKit Review Bot
Comment 8 2011-08-02 14:37:10 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.