Remove LegacyDefaultOptionalArguments flag from canvas IDL files

https://bugs.webkit.org/show_bug.cgi?id=65737

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

No new tests, all existing tests pass.

* html/canvas/CanvasGradient.idl:
* html/canvas/Float32Array.idl:
* html/canvas/Float64Array.idl:
* html/canvas/Int16Array.idl:
* html/canvas/Int32Array.idl:
* html/canvas/Int8Array.idl:
* html/canvas/OESVertexArrayObject.idl:
* html/canvas/Uint16Array.idl:
* html/canvas/Uint32Array.idl:
* html/canvas/Uint8Array.idl:

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

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 0587138244511de02e1e31a2a3084a6dee0ff10a..0496f351cc4e5743c6ca642271b738f0cdbd5c64 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,23 @@
+2011-08-04  Mark Pilgrim  <pilgrim@chromium.org>
+
+        Remove LegacyDefaultOptionalArguments flag from canvas IDL files
+        https://bugs.webkit.org/show_bug.cgi?id=65737
+
+        Reviewed by Adam Barth.
+
+        No new tests, all existing tests pass.
+
+        * html/canvas/CanvasGradient.idl:
+        * html/canvas/Float32Array.idl:
+        * html/canvas/Float64Array.idl:
+        * html/canvas/Int16Array.idl:
+        * html/canvas/Int32Array.idl:
+        * html/canvas/Int8Array.idl:
+        * html/canvas/OESVertexArrayObject.idl:
+        * html/canvas/Uint16Array.idl:
+        * html/canvas/Uint32Array.idl:
+        * html/canvas/Uint8Array.idl:
+
 2011-08-04  MORITA Hajime  <morrita@google.com>
 
         Reviewed by Ryosuke Niwa.

diff --git a/Source/WebCore/html/canvas/CanvasGradient.idl b/Source/WebCore/html/canvas/CanvasGradient.idl
index 31d85a3da9fa0e0cd51d539ab3229584e47269be..48f75bbdab7684d5916a8bcbe9cd42b43f0dd4e3 100644 (file)
--- a/Source/WebCore/html/canvas/CanvasGradient.idl
+++ b/Source/WebCore/html/canvas/CanvasGradient.idl
@@ -26,12 +26,12 @@
 module html {
 
     interface [
-        LegacyDefaultOptionalArguments,
         InterfaceUUID=bb1108ea-6b8c-4a08-894a-218628630cdb,
         ImplementationUUID=a2942ae6-2731-4286-98cc-9d5e79e20de1
     ] CanvasGradient {
 
-        void addColorStop(in float offset, in DOMString color)
+        void addColorStop(in [Optional=CallWithDefaultValue] float offset, 
+                          in [Optional=CallWithDefaultValue] DOMString color)
             raises (DOMException);
 
     };

diff --git a/Source/WebCore/html/canvas/Float32Array.idl b/Source/WebCore/html/canvas/Float32Array.idl
index 1fb43efbff1826c962ccdfc11fd2b3fcc13f5007..3ba498305c6b153c1ad9f1939291adf100dc2259 100644 (file)
--- a/Source/WebCore/html/canvas/Float32Array.idl
+++ b/Source/WebCore/html/canvas/Float32Array.idl
@@ -26,7 +26,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -40,7 +39,8 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 4;
 
         readonly attribute unsigned long length;
-        Float32Array subarray(in long start, in [Optional] long end);
+        Float32Array subarray(in [Optional=CallWithDefaultValue] long start, 
+                              in [Optional] long end);
 
         // void set(in Float32Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);

diff --git a/Source/WebCore/html/canvas/Float64Array.idl b/Source/WebCore/html/canvas/Float64Array.idl
index c55bec3e2818bcb7f1de76f9a7b5b1037d25c694..3fac4622810fa451da40488d45a9676eaff459e0 100644 (file)
--- a/Source/WebCore/html/canvas/Float64Array.idl
+++ b/Source/WebCore/html/canvas/Float64Array.idl
@@ -26,7 +26,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -40,7 +39,8 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 8;
 
         readonly attribute unsigned long length;
-        Float64Array subarray(in long start, in [Optional] long end);
+        Float64Array subarray(in [Optional=CallWithDefaultValue] long start, 
+                              in [Optional] long end);
 
         // void set(in Float64Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);

diff --git a/Source/WebCore/html/canvas/Int16Array.idl b/Source/WebCore/html/canvas/Int16Array.idl
index 971e988035afc810e8acedf0dd1f2338dad1fae0..81de4ed42cbf61eda3647b5e18bf587aa2f0e23d 100644 (file)
--- a/Source/WebCore/html/canvas/Int16Array.idl
+++ b/Source/WebCore/html/canvas/Int16Array.idl
@@ -25,7 +25,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -39,7 +38,8 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 2;
 
         readonly attribute unsigned long length;
-        Int16Array subarray(in long start, in [Optional] long end);
+        Int16Array subarray(in [Optional=CallWithDefaultValue] long start, 
+                            in [Optional] long end);
 
         // void set(in Int16Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);

diff --git a/Source/WebCore/html/canvas/Int32Array.idl b/Source/WebCore/html/canvas/Int32Array.idl
index a073653bb7a8615c3e944cba4c84b3f8549c2002..0d7a4f4de084f1d32d7cb9587badfcdf7359c523 100644 (file)
--- a/Source/WebCore/html/canvas/Int32Array.idl
+++ b/Source/WebCore/html/canvas/Int32Array.idl
@@ -26,7 +26,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -40,7 +39,8 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 4;
 
         readonly attribute unsigned long length;
-        Int32Array subarray(in long start, in [Optional] long end);
+        Int32Array subarray(in [Optional=CallWithDefaultValue] long start, 
+                            in [Optional] long end);
 
         // void set(in Int32Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);

diff --git a/Source/WebCore/html/canvas/Int8Array.idl b/Source/WebCore/html/canvas/Int8Array.idl
index bf1989fbc669d9f6689a161c25e0a39448abdb5b..c13456bdcb231a2ebc2609ba75216bfd742ef1b1 100644 (file)
--- a/Source/WebCore/html/canvas/Int8Array.idl
+++ b/Source/WebCore/html/canvas/Int8Array.idl
@@ -26,7 +26,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -40,7 +39,8 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 1;
 
         readonly attribute unsigned long length;
-        Int8Array subarray(in long start, in [Optional] long end);
+        Int8Array subarray(in [Optional=CallWithDefaultValue] long start, 
+                           in [Optional] long end);
 
         // void set(in Int8Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);

diff --git a/Source/WebCore/html/canvas/OESVertexArrayObject.idl b/Source/WebCore/html/canvas/OESVertexArrayObject.idl
index 3e9f8badcb646bdd0e0fda626c84b93585f982dd..cfb6506bb07646d4e9850e66b515ec08223d3c57 100644 (file)
--- a/Source/WebCore/html/canvas/OESVertexArrayObject.idl
+++ b/Source/WebCore/html/canvas/OESVertexArrayObject.idl
@@ -26,7 +26,6 @@
 module html {
     interface [
         Conditional=WEBGL, 
-        LegacyDefaultOptionalArguments,
         GenerateIsReachable=ImplContext,
         OmitConstructor, 
         DontCheckEnums
@@ -34,8 +33,8 @@ module html {
         const unsigned int VERTEX_ARRAY_BINDING_OES = 0x85B5;
         
         [StrictTypeChecking] WebGLVertexArrayObjectOES createVertexArrayOES();
-        [StrictTypeChecking] void         deleteVertexArrayOES(in WebGLVertexArrayObjectOES arrayObject);
-        [StrictTypeChecking] boolean      isVertexArrayOES(in WebGLVertexArrayObjectOES arrayObject);
-        [StrictTypeChecking] void         bindVertexArrayOES(in WebGLVertexArrayObjectOES arrayObject) raises(DOMException);
+        [StrictTypeChecking] void         deleteVertexArrayOES(in [Optional=CallWithDefaultValue] WebGLVertexArrayObjectOES arrayObject);
+        [StrictTypeChecking] boolean      isVertexArrayOES(in [Optional=CallWithDefaultValue] WebGLVertexArrayObjectOES arrayObject);
+        [StrictTypeChecking] void         bindVertexArrayOES(in [Optional=CallWithDefaultValue] WebGLVertexArrayObjectOES arrayObject) raises(DOMException);
     };
 }

diff --git a/Source/WebCore/html/canvas/Uint16Array.idl b/Source/WebCore/html/canvas/Uint16Array.idl
index 5f20f4559f5cd752b3efd7461ee93af2becba44f..0c8ac230e9034b59acf644af0510c3742c7c5995 100644 (file)
--- a/Source/WebCore/html/canvas/Uint16Array.idl
+++ b/Source/WebCore/html/canvas/Uint16Array.idl
@@ -26,7 +26,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -40,7 +39,7 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 2;
 
         readonly attribute unsigned long length;
-        Uint16Array subarray(in long start, in [Optional] long end);
+        Uint16Array subarray(in [Optional=CallWithDefaultValue] long start, in [Optional] long end);
 
         // void set(in Uint16Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);

diff --git a/Source/WebCore/html/canvas/Uint32Array.idl b/Source/WebCore/html/canvas/Uint32Array.idl
index a560bf5d93e44686c0a41fec2f2861bef33a12cc..001b6dde30016e6cda76ad5767dd436eae1a7283 100644 (file)
--- a/Source/WebCore/html/canvas/Uint32Array.idl
+++ b/Source/WebCore/html/canvas/Uint32Array.idl
@@ -26,7 +26,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -40,7 +39,7 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 4;
 
         readonly attribute unsigned long length;
-        Uint32Array subarray(in long start, in [Optional] long end);
+        Uint32Array subarray(in [Optional=CallWithDefaultValue] long start, in [Optional] long end);
 
         // void set(in Uint32Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);

diff --git a/Source/WebCore/html/canvas/Uint8Array.idl b/Source/WebCore/html/canvas/Uint8Array.idl
index b8402105b7af12661b9df8224886322ffb5a7281..53a1ec618c1ecd4442a31504cd4c1a73e74e300a 100644 (file)
--- a/Source/WebCore/html/canvas/Uint8Array.idl
+++ b/Source/WebCore/html/canvas/Uint8Array.idl
@@ -26,7 +26,6 @@
 
 module html {
     interface [
-        LegacyDefaultOptionalArguments,
         CanBeConstructed,
         CustomConstructFunction,
         V8CustomConstructor,
@@ -40,7 +39,7 @@ module html {
         const unsigned int BYTES_PER_ELEMENT = 1;
 
         readonly attribute unsigned long length;
-        Uint8Array subarray(in long start, in [Optional] long end);
+        Uint8Array subarray(in [Optional=CallWithDefaultValue] long start, in [Optional] long end);
 
         // void set(in Uint8Array array, [Optional] in unsigned long offset);
         // void set(in sequence<long> array, [Optional] in unsigned long offset);