Title: [122545] trunk/Source/WebCore
Revision
122545
Author
[email protected]
Date
2012-07-12 23:50:15 -0700 (Thu, 12 Jul 2012)

Log Message

CodeGeneratorJS.pm need to handle the attribute which has "CallWith=ScriptExecutionContext" option.
https://bugs.webkit.org/show_bug.cgi?id=91185

Patch by Dongwoo Im <[email protected]> on 2012-07-12
Reviewed by Kentaro Hara.

When an attribute has "CallWith=ScriptExecutionContext" option, 'ScriptExecutionContext*' parameter should be the last parameter.

No new tests. Covered by existing tests.

* bindings/scripts/CodeGeneratorJS.pm:
(GenerateImplementation):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (122544 => 122545)


--- trunk/Source/WebCore/ChangeLog	2012-07-13 06:41:18 UTC (rev 122544)
+++ trunk/Source/WebCore/ChangeLog	2012-07-13 06:50:15 UTC (rev 122545)
@@ -1,3 +1,17 @@
+2012-07-12  Dongwoo Im  <[email protected]>
+
+        CodeGeneratorJS.pm need to handle the attribute which has "CallWith=ScriptExecutionContext" option.
+        https://bugs.webkit.org/show_bug.cgi?id=91185
+
+        Reviewed by Kentaro Hara.
+
+        When an attribute has "CallWith=ScriptExecutionContext" option, 'ScriptExecutionContext*' parameter should be the last parameter.
+
+        No new tests. Covered by existing tests.
+
+        * bindings/scripts/CodeGeneratorJS.pm:
+        (GenerateImplementation):
+
 2012-07-12  Yoshifumi Inoue  <[email protected]>
 
         REGRESSION(r117738): [Forms] stepMismatch for input type "time" with large step value always return false.

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm (122544 => 122545)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2012-07-13 06:41:18 UTC (rev 122544)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2012-07-13 06:50:15 UTC (rev 122545)
@@ -1796,7 +1796,7 @@
                             $functionName = "impl->${functionName}";
                         }
 
-                        unshift(@arguments, @callWithArgs);
+                        push(@arguments, @callWithArgs);
 
                         my $jsType = NativeToJSValue($attribute->signature, 0, $implClassName, "${functionName}(" . join(", ", @arguments) . ")", "castedThis");
                         push(@implContent, "    $implClassName* impl = static_cast<$implClassName*>(castedThis->impl());\n");
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to