Remove LegacyDefaultOptionalArguments flag from CSS IDL files
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Aug 2011 21:35:15 +0000 (21:35 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Aug 2011 21:35:15 +0000 (21:35 +0000)
https://bugs.webkit.org/show_bug.cgi?id=65341

Patch by Mark Pilgrim <pilgrim@chromium.org> on 2011-08-02
Reviewed by Adam Barth.

No new tests. All existing tests pass.

* css/CSSMediaRule.idl:
* css/CSSPrimitiveValue.idl:
* css/CSSRuleList.idl:
* css/CSSStyleDeclaration.idl:
* css/CSSStyleSheet.idl:
* css/CSSValueList.idl:
* css/MediaList.idl:
* css/MediaQueryList.idl:
* css/MediaQueryListListener.idl:
* css/StyleMedia.idl:
* css/StyleSheetList.idl:
* css/WebKitCSSKeyframesRule.idl:
* css/WebKitCSSMatrix.idl:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@92236 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 files changed:
Source/WebCore/ChangeLog
Source/WebCore/css/CSSMediaRule.idl
Source/WebCore/css/CSSPrimitiveValue.idl
Source/WebCore/css/CSSRuleList.idl
Source/WebCore/css/CSSStyleDeclaration.idl
Source/WebCore/css/CSSStyleSheet.idl
Source/WebCore/css/CSSValueList.idl
Source/WebCore/css/MediaList.idl
Source/WebCore/css/MediaQueryList.idl
Source/WebCore/css/MediaQueryListListener.idl
Source/WebCore/css/StyleMedia.idl
Source/WebCore/css/StyleSheetList.idl
Source/WebCore/css/WebKitCSSKeyframesRule.idl
Source/WebCore/css/WebKitCSSMatrix.idl

index eba6cc7..614dc8e 100644 (file)
@@ -1,3 +1,26 @@
+2011-08-02  Mark Pilgrim  <pilgrim@chromium.org>
+
+        Remove LegacyDefaultOptionalArguments flag from CSS IDL files
+        https://bugs.webkit.org/show_bug.cgi?id=65341
+
+        Reviewed by Adam Barth.
+
+        No new tests. All existing tests pass.
+
+        * css/CSSMediaRule.idl:
+        * css/CSSPrimitiveValue.idl:
+        * css/CSSRuleList.idl:
+        * css/CSSStyleDeclaration.idl:
+        * css/CSSStyleSheet.idl:
+        * css/CSSValueList.idl:
+        * css/MediaList.idl:
+        * css/MediaQueryList.idl:
+        * css/MediaQueryListListener.idl:
+        * css/StyleMedia.idl:
+        * css/StyleSheetList.idl:
+        * css/WebKitCSSKeyframesRule.idl:
+        * css/WebKitCSSMatrix.idl:
+
 2011-08-02  Robert Kroeger  <rjkroege@chromium.org>
 
         [chromium] Layering violations in gesture recognizer
index ecbaf63..cf1a488 100644 (file)
 module css {
 
     // Introduced in DOM Level 2:
-    interface [
-        LegacyDefaultOptionalArguments,
-    ] CSSMediaRule : CSSRule {
+    interface CSSMediaRule : CSSRule {
         readonly attribute MediaList media;
         readonly attribute CSSRuleList cssRules;
         
-        [OldStyleObjC] unsigned long      insertRule(in DOMString rule, 
-                                                     in unsigned long index)
+        [OldStyleObjC] unsigned long      insertRule(in [Optional=CallWithDefaultValue] DOMString rule, 
+                                                     in [Optional=CallWithDefaultValue] unsigned long index)
             raises(DOMException);
-        void               deleteRule(in unsigned long index)
+        void               deleteRule(in [Optional=CallWithDefaultValue] unsigned long index)
             raises(DOMException);
     };
 
index 957cb0f..04f1704 100644 (file)
@@ -19,9 +19,7 @@
 
 module css {
 
-    interface [
-        LegacyDefaultOptionalArguments,
-    ] CSSPrimitiveValue : CSSValue {
+    interface CSSPrimitiveValue : CSSValue {
 
         // UnitTypes
         const unsigned short CSS_UNKNOWN    = 0;
@@ -53,13 +51,13 @@ module css {
 
         readonly attribute unsigned short primitiveType;
 
-        [OldStyleObjC] void setFloatValue(in unsigned short unitType,
-                                          in float floatValue)
+        [OldStyleObjC] void setFloatValue(in [Optional=CallWithDefaultValue] unsigned short unitType,
+                                          in [Optional=CallWithDefaultValue] float floatValue)
             raises(DOMException);
-        float getFloatValue(in unsigned short unitType)
+        float getFloatValue(in [Optional=CallWithDefaultValue] unsigned short unitType)
             raises(DOMException);
-        [OldStyleObjC] void setStringValue(in unsigned short stringType, 
-                                           in DOMString stringValue)
+        [OldStyleObjC] void setStringValue(in [Optional=CallWithDefaultValue] unsigned short stringType, 
+                                           in [Optional=CallWithDefaultValue] DOMString stringValue)
             raises(DOMException);
         DOMString getStringValue()
             raises(DOMException);
index 4087842..ed86eaa 100644 (file)
@@ -27,13 +27,12 @@ module css {
 
     // Introduced in DOM Level 2:
     interface [
-        LegacyDefaultOptionalArguments,
         CustomIsReachable,
         HasIndexGetter,
         V8DependentLifetime
     ] CSSRuleList {
         readonly attribute unsigned long    length;
-        CSSRule           item(in unsigned long index);
+        CSSRule           item(in [Optional=CallWithDefaultValue] unsigned long index);
     };
 
 }
index c932bc4..2fa609b 100644 (file)
@@ -22,7 +22,6 @@ module css {
 
     // Introduced in DOM Level 2:
     interface [
-        LegacyDefaultOptionalArguments,
         CustomMarkFunction,
         GenerateIsReachable=ImplRoot,
         DelegatingPutFunction,
@@ -33,23 +32,23 @@ module css {
                  attribute [ConvertNullStringTo=Null, ConvertNullToNullString] DOMString        cssText
                      setter raises(DOMException);
 
-        [ConvertNullStringTo=Null] DOMString          getPropertyValue(in DOMString propertyName);
-        [JSCCustom] CSSValue           getPropertyCSSValue(in DOMString propertyName);
-        [ConvertNullStringTo=Null] DOMString          removeProperty(in DOMString propertyName)
+        [ConvertNullStringTo=Null] DOMString          getPropertyValue(in [Optional=CallWithDefaultValue] DOMString propertyName);
+        [JSCCustom] CSSValue           getPropertyCSSValue(in [Optional=CallWithDefaultValue] DOMString propertyName);
+        [ConvertNullStringTo=Null] DOMString          removeProperty(in [Optional=CallWithDefaultValue] DOMString propertyName)
             raises(DOMException);
-        [ConvertNullStringTo=Null] DOMString          getPropertyPriority(in DOMString propertyName);
-        [OldStyleObjC] void setProperty(in DOMString propertyName, 
-                                        in [ConvertNullToNullString] DOMString value, 
-                                        in DOMString priority)
+        [ConvertNullStringTo=Null] DOMString          getPropertyPriority(in [Optional=CallWithDefaultValue] DOMString propertyName);
+        [OldStyleObjC] void setProperty(in [Optional=CallWithDefaultValue] DOMString propertyName, 
+                                        in [ConvertNullToNullString,Optional=CallWithDefaultValue] DOMString value, 
+                                        in [Optional=CallWithDefaultValue] DOMString priority)
             raises(DOMException);
 
         readonly attribute unsigned long    length;
-        DOMString          item(in unsigned long index);
+        DOMString          item(in [Optional=CallWithDefaultValue] unsigned long index);
         readonly attribute CSSRule          parentRule;
 
         // Extensions
-        [ConvertNullStringTo=Null] DOMString          getPropertyShorthand(in DOMString propertyName);
-        boolean            isPropertyImplicit(in DOMString propertyName);
+        [ConvertNullStringTo=Null] DOMString          getPropertyShorthand(in [Optional=CallWithDefaultValue] DOMString propertyName);
+        boolean            isPropertyImplicit(in [Optional=CallWithDefaultValue] DOMString propertyName);
     };
 
 }
index 571567d..3497827 100644 (file)
 module css {
 
     // Introduced in DOM Level 2:
-    interface [
-        LegacyDefaultOptionalArguments,
-    ] CSSStyleSheet : StyleSheet {
+    interface CSSStyleSheet : StyleSheet {
         readonly attribute CSSRule          ownerRule;
         readonly attribute CSSRuleList      cssRules;
 
-        [OldStyleObjC] unsigned long insertRule(in DOMString rule, 
-                                                in unsigned long index)
+        [OldStyleObjC] unsigned long insertRule(in [Optional=CallWithDefaultValue] DOMString rule, 
+                                                in [Optional=CallWithDefaultValue] unsigned long index)
             raises(DOMException);
-        void               deleteRule(in unsigned long index)
+        void               deleteRule(in [Optional=CallWithDefaultValue] unsigned long index)
             raises(DOMException);
 
         // IE Extensions
         readonly attribute CSSRuleList      rules;
 
-        long addRule(in DOMString selector,
-                     in DOMString style,
+        long addRule(in [Optional=CallWithDefaultValue] DOMString selector,
+                     in [Optional=CallWithDefaultValue] DOMString style,
                      in [Optional] unsigned long index)
             raises(DOMException);
-        void removeRule(in unsigned long index)
+        void removeRule(in [Optional=CallWithDefaultValue] unsigned long index)
             raises(DOMException);
     };
 
index 281a11b..63c8514 100644 (file)
@@ -27,11 +27,10 @@ module css {
 
     // Introduced in DOM Level 2:
     interface [
-        LegacyDefaultOptionalArguments,
         HasIndexGetter
     ] CSSValueList : CSSValue {
         readonly attribute unsigned long    length;
-        CSSValue           item(in unsigned long index);
+        CSSValue           item(in [Optional=CallWithDefaultValue] unsigned long index);
     };
 
 }
index 3e9fa22..2b441f0 100644 (file)
@@ -27,7 +27,6 @@ module stylesheets {
 
     // Introduced in DOM Level 2:
     interface [
-        LegacyDefaultOptionalArguments,
         GenerateIsReachable,
         HasIndexGetter
     ] MediaList {
@@ -36,10 +35,10 @@ module stylesheets {
                      setter raises(DOMException);
         readonly attribute unsigned long length;
 
-        [ConvertNullStringTo=Null] DOMString item(in unsigned long index);
-        void deleteMedium(in DOMString oldMedium)
+        [ConvertNullStringTo=Null] DOMString item(in [Optional=CallWithDefaultValue] unsigned long index);
+        void deleteMedium(in [Optional=CallWithDefaultValue] DOMString oldMedium)
             raises(DOMException);
-        void appendMedium(in DOMString newMedium)
+        void appendMedium(in [Optional=CallWithDefaultValue] DOMString newMedium)
             raises(DOMException);
 
     };
index 74db56e..0d8b1f2 100644 (file)
  */
 
 module view {
-    interface [
-        LegacyDefaultOptionalArguments,
-    ] MediaQueryList {
+    interface MediaQueryList {
         readonly attribute DOMString media;
         readonly attribute boolean matches;
-        void addListener(in MediaQueryListListener listener);
-        void removeListener(in MediaQueryListListener listener);
+        void addListener(in [Optional=CallWithDefaultValue] MediaQueryListListener listener);
+        void removeListener(in [Optional=CallWithDefaultValue] MediaQueryListListener listener);
     };
 }
index 5b773cd..dfe659b 100644 (file)
 
 module view {
     interface [
-        LegacyDefaultOptionalArguments,
         NoStaticTables,
         ObjCProtocol,
         PureInterface,
         OmitConstructor
     ] MediaQueryListListener {
-        void queryChanged(in MediaQueryList list);
+        void queryChanged(in [Optional=CallWithDefaultValue] MediaQueryList list);
     };
 }
index dabd21c..09f2ae1 100644 (file)
 
 module view {
     interface [
-        LegacyDefaultOptionalArguments,
         GenerateIsReachable=ImplFrame
     ] StyleMedia {
         readonly attribute DOMString type;
-        boolean matchMedium(in DOMString mediaquery);
+        boolean matchMedium(in [Optional=CallWithDefaultValue] DOMString mediaquery);
     };
 }
index 0dacf98..a7048d9 100644 (file)
@@ -22,14 +22,13 @@ module stylesheets {
 
     // Introduced in DOM Level 2:
     interface [
-        LegacyDefaultOptionalArguments,
         GenerateIsReachable=ImplDocument,
         HasIndexGetter,
         HasNameGetter,
         V8DependentLifetime
     ] StyleSheetList {
         readonly attribute unsigned long    length;
-        StyleSheet         item(in unsigned long index);
+        StyleSheet         item(in [Optional=CallWithDefaultValue] unsigned long index);
     };
 
 }
index 7058b0a..3b08130 100644 (file)
@@ -30,16 +30,15 @@ module css {
 
     // Introduced in DOM Level ?:
     interface [
-        LegacyDefaultOptionalArguments,
         HasIndexGetter
     ] WebKitCSSKeyframesRule : CSSRule {
 
         attribute [ConvertNullStringTo=Null, ConvertNullToNullString] DOMString name;
         readonly attribute CSSRuleList cssRules;
         
-        void insertRule(in DOMString rule);
-        void deleteRule(in DOMString key);
-        WebKitCSSKeyframeRule findRule(in DOMString key);
+        void insertRule(in [Optional=CallWithDefaultValue] DOMString rule);
+        void deleteRule(in [Optional=CallWithDefaultValue] DOMString key);
+        WebKitCSSKeyframeRule findRule(in [Optional=CallWithDefaultValue] DOMString key);
     };
 
 }
index 00fd976..76ff601 100644 (file)
@@ -27,7 +27,6 @@ module css {
 
     // Introduced in DOM Level ?:
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         ConstructorParameters=1,
@@ -59,40 +58,49 @@ module css {
         attribute double m43;
         attribute double m44;
 
-        void setMatrixValue(in DOMString string) raises (DOMException);
+        void setMatrixValue(in [Optional=CallWithDefaultValue] DOMString string) raises (DOMException);
         
         // Multiply this matrix by secondMatrix, on the right (result = this * secondMatrix)
-        [Immutable] WebKitCSSMatrix multiply(in WebKitCSSMatrix secondMatrix);
+        [Immutable] WebKitCSSMatrix multiply(in [Optional=CallWithDefaultValue] WebKitCSSMatrix secondMatrix);
         
         // Return the inverse of this matrix. Throw an exception if the matrix is not invertible
         [Immutable] WebKitCSSMatrix inverse() raises (DOMException);
         
         // Return this matrix translated by the passed values.
         // Passing a NaN will use a value of 0. This allows the 3D form to used for 2D operations    
-        [Immutable] WebKitCSSMatrix translate(in double x, in double y, in double z);
+        [Immutable] WebKitCSSMatrix translate(in [Optional=CallWithDefaultValue] double x, 
+                                              in [Optional=CallWithDefaultValue] double y, 
+                                              in [Optional=CallWithDefaultValue] double z);
         
         // Returns this matrix scaled by the passed values.
         // Passing scaleX or scaleZ as NaN uses a value of 1, but passing scaleY of NaN 
         // makes it the same as scaleX. This allows the 3D form to used for 2D operations
-        [Immutable] WebKitCSSMatrix scale(in double scaleX, in double scaleY, in double scaleZ);
+        [Immutable] WebKitCSSMatrix scale(in [Optional=CallWithDefaultValue] double scaleX, 
+                                          in [Optional=CallWithDefaultValue] double scaleY, 
+                                          in [Optional=CallWithDefaultValue] double scaleZ);
         
         // Returns this matrix rotated by the passed values.
         // If rotY and rotZ are NaN, rotate about Z (rotX=0, rotateY=0, rotateZ=rotX).
         // Otherwise use a rotation value of 0 for any passed NaN.    
-        [Immutable] WebKitCSSMatrix rotate(in double rotX, in double rotY, in double rotZ);
+        [Immutable] WebKitCSSMatrix rotate(in [Optional=CallWithDefaultValue] double rotX, 
+                                           in [Optional=CallWithDefaultValue] double rotY, 
+                                           in [Optional=CallWithDefaultValue] double rotZ);
         
         // Returns this matrix rotated about the passed axis by the passed angle.
         // Passing a NaN will use a value of 0. If the axis is (0,0,0) use a value
         // of (0,0,1).
-        [Immutable] WebKitCSSMatrix rotateAxisAngle(in double x, in double y, in double z, in double angle);
+        [Immutable] WebKitCSSMatrix rotateAxisAngle(in [Optional=CallWithDefaultValue] double x, 
+                                                    in [Optional=CallWithDefaultValue] double y, 
+                                                    in [Optional=CallWithDefaultValue] double z, 
+                                                    in [Optional=CallWithDefaultValue] double angle);
 
         // Returns this matrix skewed along the X axis by the passed values.
         // Passing a NaN will use a value of 0.
-        [Immutable] WebKitCSSMatrix skewX(in double angle);
+        [Immutable] WebKitCSSMatrix skewX(in [Optional=CallWithDefaultValue] double angle);
 
         // Returns this matrix skewed along the Y axis by the passed values.
         // Passing a NaN will use a value of 0.
-        [Immutable] WebKitCSSMatrix skewY(in double angle);
+        [Immutable] WebKitCSSMatrix skewY(in [Optional=CallWithDefaultValue] double angle);
 
         [DontEnum] DOMString toString();
     };