Changeset 92237 in webkit


Ignore:
Timestamp:
Aug 2, 2011 2:36:52 PM (13 years ago)
Author:
commit-queue@webkit.org
Message:

Remove LegacyDefaultOptionalArguments flag from SVG IDL files
​https://bugs.webkit.org/show_bug.cgi?id=65337

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-02
Reviewed by Nikolas Zimmermann.

No new tests, all existing tests pass.

  • svg/ElementTimeControl.idl:
  • svg/SVGAnimationElement.idl:
  • svg/SVGDocument.idl:
  • svg/SVGElementInstance.idl:
  • svg/SVGElementInstanceList.idl:
  • svg/SVGException.idl:
  • svg/SVGFEDropShadowElement.idl:
  • svg/SVGFEGaussianBlurElement.idl:
  • svg/SVGFEMorphologyElement.idl:
  • svg/SVGFilterElement.idl:
  • svg/SVGLocatable.idl:
  • svg/SVGMarkerElement.idl:
  • svg/SVGPathElement.idl:
  • svg/SVGSVGElement.idl:
  • svg/SVGStylable.idl:
  • svg/SVGStyleElement.idl:
  • svg/SVGTests.idl:
  • svg/SVGTextContentElement.idl:
Location:
trunk/Source/WebCore
Files:
19 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r92236 r92237  
     12011-08-02  Mark Pilgrim  <pilgrim@chromium.org>
     2
     3        Remove LegacyDefaultOptionalArguments flag from SVG IDL files
     4        https://bugs.webkit.org/show_bug.cgi?id=65337
     5
     6        Reviewed by Nikolas Zimmermann.
     7
     8        No new tests, all existing tests pass.
     9
     10        * svg/ElementTimeControl.idl:
     11        * svg/SVGAnimationElement.idl:
     12        * svg/SVGDocument.idl:
     13        * svg/SVGElementInstance.idl:
     14        * svg/SVGElementInstanceList.idl:
     15        * svg/SVGException.idl:
     16        * svg/SVGFEDropShadowElement.idl:
     17        * svg/SVGFEGaussianBlurElement.idl:
     18        * svg/SVGFEMorphologyElement.idl:
     19        * svg/SVGFilterElement.idl:
     20        * svg/SVGLocatable.idl:
     21        * svg/SVGMarkerElement.idl:
     22        * svg/SVGPathElement.idl:
     23        * svg/SVGSVGElement.idl:
     24        * svg/SVGStylable.idl:
     25        * svg/SVGStyleElement.idl:
     26        * svg/SVGTests.idl:
     27        * svg/SVGTextContentElement.idl:
     28
    1292011-08-02  Mark Pilgrim  <pilgrim@chromium.org>
    230

  • trunk/Source/WebCore/svg/ElementTimeControl.idl

    r89148 r92237  
    2828
    2929    interface [
    30         LegacyDefaultOptionalArguments,
    3130        Conditional=SVG,
    3231        ObjCProtocol,
    … …  
    3433    ] ElementTimeControl {
    3534        void beginElement();
    36         void beginElementAt(in float offset);
     35        void beginElementAt(in [Optional=CallWithDefaultValue] float offset);
    3736        void endElement();
    38         void endElementAt(in float offset);
     37        void endElementAt(in [Optional=CallWithDefaultValue] float offset);
    3938    };
    4039

  • trunk/Source/WebCore/svg/SVGAnimationElement.idl

    r89148 r92237  
    2727
    2828    interface [
    29         LegacyDefaultOptionalArguments,
    3029        Conditional=SVG_ANIMATION,
    3130        OmitConstructor

  • trunk/Source/WebCore/svg/SVGDocument.idl

    r89148 r92237  
    2323
    2424    interface [
    25         LegacyDefaultOptionalArguments,
    2625        Conditional=SVG
    2726    ] SVGDocument : Document {
    … …  
    2928
    3029        // Overwrite the one in events::DocumentEvent
    31         Event createEvent(in DOMString eventType)
     30        Event createEvent(in [Optional=CallWithDefaultValue] DOMString eventType)
    3231            raises(DOMException);
    3332    };

  • trunk/Source/WebCore/svg/SVGElementInstance.idl

    r89781 r92237  
    2828
    2929    interface [
    30         Conditional=SVG,
    3130        LegacyDefaultOptionalArguments,
    3231        CustomMarkFunction,

  • trunk/Source/WebCore/svg/SVGElementInstanceList.idl

    r89148 r92237  
    2626module svg {
    2727    interface [
    28         LegacyDefaultOptionalArguments,
    2928        Conditional=SVG
    3029    ] SVGElementInstanceList {
    3130        readonly attribute unsigned long length;
    3231
    33         SVGElementInstance item(in unsigned long index);
     32        SVGElementInstance item(in [Optional=CallWithDefaultValue] unsigned long index);
    3433    };
    3534}

  • trunk/Source/WebCore/svg/SVGException.idl

    r89148 r92237  
    2323    interface [
    2424        Conditional=SVG,
    25         LegacyDefaultOptionalArguments,
    2625        DontCheckEnums
    2726    ] SVGException {

  • trunk/Source/WebCore/svg/SVGFEDropShadowElement.idl

    r89148 r92237  
    2121
    2222    interface [
    23         LegacyDefaultOptionalArguments,
    2423        Conditional=SVG&FILTERS
    2524    ] SVGFEDropShadowElement : SVGElement,
    … …  
    3130        readonly attribute SVGAnimatedNumber stdDeviationY;
    3231
    33         void setStdDeviation(in float stdDeviationX, in float stdDeviationY);
     32        void setStdDeviation(in [Optional=CallWithDefaultValue] float stdDeviationX,
     33                             in [Optional=CallWithDefaultValue] float stdDeviationY);
    3434    };
    3535

  • trunk/Source/WebCore/svg/SVGFEGaussianBlurElement.idl

    r89148 r92237  
    2727
    2828    interface [
    29         LegacyDefaultOptionalArguments,
    3029        Conditional=SVG&FILTERS
    3130    ] SVGFEGaussianBlurElement : SVGElement,
    … …  
    3534        readonly attribute SVGAnimatedNumber stdDeviationY;
    3635
    37         void setStdDeviation(in float stdDeviationX, in float stdDeviationY);
     36        void setStdDeviation(in [Optional=CallWithDefaultValue] float stdDeviationX,
     37                             in [Optional=CallWithDefaultValue] float stdDeviationY);
    3838    };
    3939

  • trunk/Source/WebCore/svg/SVGFEMorphologyElement.idl

    r89148 r92237  
    2727
    2828    interface [
    29         LegacyDefaultOptionalArguments,
    3029        Conditional=SVG&FILTERS,
    3130        DontCheckEnums
    … …  
    4241        readonly attribute SVGAnimatedNumber      radiusY;
    4342
    44         void setRadius(in float radiusX, in float radiusY);
     43        void setRadius(in [Optional=CallWithDefaultValue] float radiusX,
     44                       in [Optional=CallWithDefaultValue] float radiusY);
    4545    };
    4646

  • trunk/Source/WebCore/svg/SVGFilterElement.idl

    r89148 r92237  
    2828
    2929    interface [
    30         LegacyDefaultOptionalArguments,
    3130        Conditional=SVG&FILTERS
    3231    ] SVGFilterElement : SVGElement,
    … …  
    4544        readonly attribute SVGAnimatedInteger     filterResY;
    4645
    47         void setFilterRes(in unsigned long filterResX, in unsigned long filterResY);
     46        void setFilterRes(in [Optional=CallWithDefaultValue] unsigned long filterResX,
     47                          in [Optional=CallWithDefaultValue] unsigned long filterResY);
    4848    };
    4949

  • trunk/Source/WebCore/svg/SVGLocatable.idl

    r89148 r92237  
    2828
    2929    interface [
    30         LegacyDefaultOptionalArguments,
    3130        Conditional=SVG,
    3231        ObjCProtocol,
    … …  
    3938        SVGMatrix getCTM();
    4039        SVGMatrix getScreenCTM();
    41         SVGMatrix getTransformToElement(in SVGElement element)
     40        SVGMatrix getTransformToElement(in [Optional=CallWithDefaultValue] SVGElement element)
    4241            raises(SVGException);
    4342    };

  • trunk/Source/WebCore/svg/SVGMarkerElement.idl

    r89148 r92237  
    2727
    2828    interface [
    29         LegacyDefaultOptionalArguments,
    3029        Conditional=SVG
    3130    ] SVGMarkerElement : SVGElement,
    … …  
    5352
    5453        void setOrientToAuto();
    55         void setOrientToAngle(in SVGAngle angle);
     54        void setOrientToAngle(in [Optional=CallWithDefaultValue] SVGAngle angle);
    5655    };
    5756

  • trunk/Source/WebCore/svg/SVGPathElement.idl

    r89148 r92237  
    2828
    2929    interface [
    30         LegacyDefaultOptionalArguments,
    3130        Conditional=SVG
    3231    ] SVGPathElement : SVGElement,
    … …  
    3938
    4039        float getTotalLength();
    41         SVGPoint getPointAtLength(in float distance);
    42         unsigned long getPathSegAtLength(in float distance);
     40        SVGPoint getPointAtLength(in [Optional=CallWithDefaultValue] float distance);
     41        unsigned long getPathSegAtLength(in [Optional=CallWithDefaultValue] float distance);
    4342
    4443        SVGPathSegClosePath createSVGPathSegClosePath();
    4544
    46         SVGPathSegMovetoAbs createSVGPathSegMovetoAbs(in float x,
    47                                                       in float y);
    48         SVGPathSegMovetoRel createSVGPathSegMovetoRel(in float x,
    49                                                       in float y);
     45        SVGPathSegMovetoAbs createSVGPathSegMovetoAbs(in [Optional=CallWithDefaultValue] float x,
     46                                                      in [Optional=CallWithDefaultValue] float y);
     47        SVGPathSegMovetoRel createSVGPathSegMovetoRel(in [Optional=CallWithDefaultValue] float x,
     48                                                      in [Optional=CallWithDefaultValue] float y);
    5049
    51         SVGPathSegLinetoAbs createSVGPathSegLinetoAbs(in float x,
    52                                                       in float y);
    53         SVGPathSegLinetoRel createSVGPathSegLinetoRel(in float x,
    54                                                       in float y);
     50        SVGPathSegLinetoAbs createSVGPathSegLinetoAbs(in [Optional=CallWithDefaultValue] float x,
     51                                                      in [Optional=CallWithDefaultValue] float y);
     52        SVGPathSegLinetoRel createSVGPathSegLinetoRel(in [Optional=CallWithDefaultValue] float x,
     53                                                      in [Optional=CallWithDefaultValue] float y);
    5554
    56         SVGPathSegCurvetoCubicAbs createSVGPathSegCurvetoCubicAbs(in float x,
    57                                                                   in float y,
    58                                                                   in float x1,
    59                                                                   in float y1,
    60                                                                   in float x2,
    61                                                                   in float y2);
    62         SVGPathSegCurvetoCubicRel createSVGPathSegCurvetoCubicRel(in float x,
    63                                                                   in float y,
    64                                                                   in float x1,
    65                                                                   in float y1,
    66                                                                   in float x2,
    67                                                                   in float y2);
     55        SVGPathSegCurvetoCubicAbs createSVGPathSegCurvetoCubicAbs(in [Optional=CallWithDefaultValue] float x,
     56                                                                  in [Optional=CallWithDefaultValue] float y,
     57                                                                  in [Optional=CallWithDefaultValue] float x1,
     58                                                                  in [Optional=CallWithDefaultValue] float y1,
     59                                                                  in [Optional=CallWithDefaultValue] float x2,
     60                                                                  in [Optional=CallWithDefaultValue] float y2);
     61        SVGPathSegCurvetoCubicRel createSVGPathSegCurvetoCubicRel(in [Optional=CallWithDefaultValue] float x,
     62                                                                  in [Optional=CallWithDefaultValue] float y,
     63                                                                  in [Optional=CallWithDefaultValue] float x1,
     64                                                                  in [Optional=CallWithDefaultValue] float y1,
     65                                                                  in [Optional=CallWithDefaultValue] float x2,
     66                                                                  in [Optional=CallWithDefaultValue] float y2);
    6867
    69         SVGPathSegCurvetoQuadraticAbs createSVGPathSegCurvetoQuadraticAbs(in float x,
    70                                                                           in float y,
    71                                                                           in float x1,
    72                                                                           in float y1);
    73         SVGPathSegCurvetoQuadraticRel createSVGPathSegCurvetoQuadraticRel(in float x,
    74                                                                           in float y,
    75                                                                           in float x1,
    76                                                                           in float y1);
     68        SVGPathSegCurvetoQuadraticAbs createSVGPathSegCurvetoQuadraticAbs(in [Optional=CallWithDefaultValue] float x,
     69                                                                          in [Optional=CallWithDefaultValue] float y,
     70                                                                          in [Optional=CallWithDefaultValue] float x1,
     71                                                                          in [Optional=CallWithDefaultValue] float y1);
     72        SVGPathSegCurvetoQuadraticRel createSVGPathSegCurvetoQuadraticRel(in [Optional=CallWithDefaultValue] float x,
     73                                                                          in [Optional=CallWithDefaultValue] float y,
     74                                                                          in [Optional=CallWithDefaultValue] float x1,
     75                                                                          in [Optional=CallWithDefaultValue] float y1);
    7776
    78         SVGPathSegArcAbs createSVGPathSegArcAbs(in float x,
    79                                                 in float y,
    80                                                 in float r1,
    81                                                 in float r2,
    82                                                 in float angle,
    83                                                 in boolean largeArcFlag,
    84                                                 in boolean sweepFlag);
    85         SVGPathSegArcRel createSVGPathSegArcRel(in float x,
    86                                                 in float y,
    87                                                 in float r1,
    88                                                 in float r2,
    89                                                 in float angle,
    90                                                 in boolean largeArcFlag,
    91                                                 in boolean sweepFlag);
     77        SVGPathSegArcAbs createSVGPathSegArcAbs(in [Optional=CallWithDefaultValue] float x,
     78                                                in [Optional=CallWithDefaultValue] float y,
     79                                                in [Optional=CallWithDefaultValue] float r1,
     80                                                in [Optional=CallWithDefaultValue] float r2,
     81                                                in [Optional=CallWithDefaultValue] float angle,
     82                                                in [Optional=CallWithDefaultValue] boolean largeArcFlag,
     83                                                in [Optional=CallWithDefaultValue] boolean sweepFlag);
     84        SVGPathSegArcRel createSVGPathSegArcRel(in [Optional=CallWithDefaultValue] float x,
     85                                                in [Optional=CallWithDefaultValue] float y,
     86                                                in [Optional=CallWithDefaultValue] float r1,
     87                                                in [Optional=CallWithDefaultValue] float r2,
     88                                                in [Optional=CallWithDefaultValue] float angle,
     89                                                in [Optional=CallWithDefaultValue] boolean largeArcFlag,
     90                                                in [Optional=CallWithDefaultValue] boolean sweepFlag);
    9291
    93         SVGPathSegLinetoHorizontalAbs createSVGPathSegLinetoHorizontalAbs(in float x);
    94         SVGPathSegLinetoHorizontalRel createSVGPathSegLinetoHorizontalRel(in float x);
     92        SVGPathSegLinetoHorizontalAbs createSVGPathSegLinetoHorizontalAbs(in [Optional=CallWithDefaultValue] float x);
     93        SVGPathSegLinetoHorizontalRel createSVGPathSegLinetoHorizontalRel(in [Optional=CallWithDefaultValue] float x);
    9594
    96         SVGPathSegLinetoVerticalAbs createSVGPathSegLinetoVerticalAbs(in float y);
    97         SVGPathSegLinetoVerticalRel createSVGPathSegLinetoVerticalRel(in float y);
     95        SVGPathSegLinetoVerticalAbs createSVGPathSegLinetoVerticalAbs(in [Optional=CallWithDefaultValue] float y);
     96        SVGPathSegLinetoVerticalRel createSVGPathSegLinetoVerticalRel(in [Optional=CallWithDefaultValue] float y);
    9897
    99         SVGPathSegCurvetoCubicSmoothAbs createSVGPathSegCurvetoCubicSmoothAbs(in float x,
    100                                                                               in float y,
    101                                                                               in float x2,
    102                                                                               in float y2);
    103         SVGPathSegCurvetoCubicSmoothRel createSVGPathSegCurvetoCubicSmoothRel(in float x,
    104                                                                               in float y,
    105                                                                               in float x2,
    106                                                                               in float y2);
     98        SVGPathSegCurvetoCubicSmoothAbs createSVGPathSegCurvetoCubicSmoothAbs(in [Optional=CallWithDefaultValue] float x,
     99                                                                              in [Optional=CallWithDefaultValue] float y,
     100                                                                              in [Optional=CallWithDefaultValue] float x2,
     101                                                                              in [Optional=CallWithDefaultValue] float y2);
     102        SVGPathSegCurvetoCubicSmoothRel createSVGPathSegCurvetoCubicSmoothRel(in [Optional=CallWithDefaultValue] float x,
     103                                                                              in [Optional=CallWithDefaultValue] float y,
     104                                                                              in [Optional=CallWithDefaultValue] float x2,
     105                                                                              in [Optional=CallWithDefaultValue] float y2);
    107106
    108         SVGPathSegCurvetoQuadraticSmoothAbs createSVGPathSegCurvetoQuadraticSmoothAbs(in float x,
    109                                                                                       in float y);
    110         SVGPathSegCurvetoQuadraticSmoothRel createSVGPathSegCurvetoQuadraticSmoothRel(in float x,
    111                                                                                       in float y);
     107        SVGPathSegCurvetoQuadraticSmoothAbs createSVGPathSegCurvetoQuadraticSmoothAbs(in [Optional=CallWithDefaultValue] float x,
     108                                                                                      in [Optional=CallWithDefaultValue] float y);
     109        SVGPathSegCurvetoQuadraticSmoothRel createSVGPathSegCurvetoQuadraticSmoothRel(in [Optional=CallWithDefaultValue] float x,
     110                                                                                      in [Optional=CallWithDefaultValue] float y);
    112111
    113112        readonly attribute SVGPathSegList pathSegList;

  • trunk/Source/WebCore/svg/SVGSVGElement.idl

    r89148 r92237  
    2727    // TODO: no events::DocumentEvent available!
    2828    interface [
    29         LegacyDefaultOptionalArguments,
    3029        Conditional=SVG
    3130    ] SVGSVGElement : SVGElement,
    … …  
    5756        readonly attribute SVGPoint currentTranslate;
    5857
    59         unsigned long suspendRedraw(in unsigned long maxWaitMilliseconds);
    60         void unsuspendRedraw(in unsigned long suspendHandleId);
     58        unsigned long suspendRedraw(in [Optional=CallWithDefaultValue] unsigned long maxWaitMilliseconds);
     59        void unsuspendRedraw(in [Optional=CallWithDefaultValue] unsigned long suspendHandleId);
    6160        void unsuspendRedrawAll();
    6261        void forceRedraw();
    … …  
    6564        boolean animationsPaused();
    6665        float getCurrentTime();
    67         void setCurrentTime(in float seconds);
    68         NodeList getIntersectionList(in SVGRect rect,
    69                                            in SVGElement referenceElement);
    70         NodeList getEnclosureList(in SVGRect rect,
    71                                         in SVGElement referenceElement);
    72         boolean checkIntersection(in SVGElement element,
    73                                   in SVGRect rect);
    74         boolean checkEnclosure(in SVGElement element,
    75                                in SVGRect rect);
     66        void setCurrentTime(in [Optional=CallWithDefaultValue] float seconds);
     67        NodeList getIntersectionList(in [Optional=CallWithDefaultValue] SVGRect rect,
     68                                     in [Optional=CallWithDefaultValue] SVGElement referenceElement);
     69        NodeList getEnclosureList(in [Optional=CallWithDefaultValue] SVGRect rect,
     70                                  in [Optional=CallWithDefaultValue] SVGElement referenceElement);
     71        boolean checkIntersection(in [Optional=CallWithDefaultValue] SVGElement element,
     72                                  in [Optional=CallWithDefaultValue] SVGRect rect);
     73        boolean checkEnclosure(in [Optional=CallWithDefaultValue] SVGElement element,
     74                               in [Optional=CallWithDefaultValue] SVGRect rect);
    7675        void deselectAll();
    7776
    … …  
    8382        SVGRect createSVGRect();
    8483        SVGTransform createSVGTransform();
    85         SVGTransform createSVGTransformFromMatrix(in SVGMatrix matrix);
    86         Element getElementById(in DOMString elementId);
     84        SVGTransform createSVGTransformFromMatrix(in [Optional=CallWithDefaultValue] SVGMatrix matrix);
     85        Element getElementById(in [Optional=CallWithDefaultValue] DOMString elementId);
    8786    };
    8887

  • trunk/Source/WebCore/svg/SVGStylable.idl

    r89148 r92237  
    2929
    3030    interface [
    31         LegacyDefaultOptionalArguments,
    3231        Conditional=SVG,
    3332        ObjCProtocol,
    … …  
    3736        readonly attribute CSSStyleDeclaration style;
    3837
    39         CSSValue getPresentationAttribute(in DOMString name);
     38        CSSValue getPresentationAttribute(in [Optional=CallWithDefaultValue] DOMString name);
    4039    };
    4140

  • trunk/Source/WebCore/svg/SVGStyleElement.idl

    r89148 r92237  
    2828
    2929    interface [
    30         LegacyDefaultOptionalArguments,
    3130        Conditional=SVG
    3231    ] SVGStyleElement : SVGElement,

  • trunk/Source/WebCore/svg/SVGTests.idl

    r89148 r92237  
    2828
    2929    interface [
    30         LegacyDefaultOptionalArguments,
    3130        Conditional=SVG,
    3231        ObjCProtocol,
    … …  
    3736        readonly attribute SVGStringList systemLanguage;
    3837
    39         boolean hasExtension(in DOMString extension);
     38        boolean hasExtension(in [Optional=CallWithDefaultValue] DOMString extension);
    4039    };
    4140

  • trunk/Source/WebCore/svg/SVGTextContentElement.idl

    r89148 r92237  
    2727
    2828    interface [
    29         LegacyDefaultOptionalArguments,
    3029        Conditional=SVG
    3130    ] SVGTextContentElement : SVGElement,
    … …  
    4443        long getNumberOfChars();
    4544        float getComputedTextLength();
    46         float getSubStringLength(in [IsIndex] unsigned long offset,
    47                                  in [IsIndex] unsigned long length)
     45        float getSubStringLength(in [Optional=CallWithDefaultValue,IsIndex] unsigned long offset,
     46                                 in [Optional=CallWithDefaultValue,IsIndex] unsigned long length)
    4847            raises(DOMException);
    49         SVGPoint getStartPositionOfChar(in [IsIndex] unsigned long offset)
     48        SVGPoint getStartPositionOfChar(in [Optional=CallWithDefaultValue,IsIndex] unsigned long offset)
    5049            raises(DOMException);
    51         SVGPoint getEndPositionOfChar(in [IsIndex] unsigned long offset)
     50        SVGPoint getEndPositionOfChar(in [Optional=CallWithDefaultValue,IsIndex] unsigned long offset)
    5251            raises(DOMException);
    53         SVGRect getExtentOfChar(in [IsIndex] unsigned long offset)
     52        SVGRect getExtentOfChar(in [Optional=CallWithDefaultValue,IsIndex] unsigned long offset)
    5453            raises(DOMException);
    55         float getRotationOfChar(in [IsIndex] unsigned long offset)
     54        float getRotationOfChar(in [Optional=CallWithDefaultValue,IsIndex] unsigned long offset)
    5655            raises(DOMException);
    57         long getCharNumAtPosition(in SVGPoint point);
    58         void selectSubString(in [IsIndex] unsigned long offset,
    59                              in [IsIndex] unsigned long length)
     56        long getCharNumAtPosition(in [Optional=CallWithDefaultValue] SVGPoint point);
     57        void selectSubString(in [Optional=CallWithDefaultValue,IsIndex] unsigned long offset,
     58                             in [Optional=CallWithDefaultValue,IsIndex] unsigned long length)
    6059            raises(DOMException);
    6160    };
Note: See TracChangeset for help on using the changeset viewer.