Title: [165880] releases/WebKitGTK/webkit-2.2/Source/WebCore
Revision
165880
Author
[email protected]
Date
2014-03-19 01:49:55 -0700 (Wed, 19 Mar 2014)

Log Message

Merge r164980 - [GTK] webkit_dom_range_compare_boundary_points fails when 0 is passed as how parameter
https://bugs.webkit.org/show_bug.cgi?id=129145

Patch by Tomas Popela <[email protected]> on 2014-03-03
Reviewed by Carlos Garcia Campos.

While generating GObject DOM API the generator should skip non pointer
types while generating the g_return_val_if_fail macro.
This will avoid the situation when Range.START_TO_START is defined as 0,
thus the webkit_dom_range_compare_boundary_points function will fail
everytime it is called with 0 as how value.

* bindings/scripts/CodeGeneratorGObject.pm:
(GenerateFunction):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.2/Source/WebCore/ChangeLog (165879 => 165880)


--- releases/WebKitGTK/webkit-2.2/Source/WebCore/ChangeLog	2014-03-19 08:38:28 UTC (rev 165879)
+++ releases/WebKitGTK/webkit-2.2/Source/WebCore/ChangeLog	2014-03-19 08:49:55 UTC (rev 165880)
@@ -1,3 +1,19 @@
+2014-03-03  Tomas Popela  <[email protected]>
+
+        [GTK] webkit_dom_range_compare_boundary_points fails when 0 is passed as how parameter
+        https://bugs.webkit.org/show_bug.cgi?id=129145
+
+        Reviewed by Carlos Garcia Campos.
+
+        While generating GObject DOM API the generator should skip non pointer
+        types while generating the g_return_val_if_fail macro.
+        This will avoid the situation when Range.START_TO_START is defined as 0,
+        thus the webkit_dom_range_compare_boundary_points function will fail
+        everytime it is called with 0 as how value.
+
+        * bindings/scripts/CodeGeneratorGObject.pm:
+        (GenerateFunction):
+
 2014-02-11  Piotr Grad  <[email protected]>
 
         [GStreamer] High playback rate causes crash

Modified: releases/WebKitGTK/webkit-2.2/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm (165879 => 165880)


--- releases/WebKitGTK/webkit-2.2/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm	2014-03-19 08:38:28 UTC (rev 165879)
+++ releases/WebKitGTK/webkit-2.2/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm	2014-03-19 08:49:55 UTC (rev 165880)
@@ -971,9 +971,8 @@
     foreach my $param (@{$function->parameters}) {
         my $paramName = $param->name;
         my $paramIDLType = $param->type;
-        my $paramTypeIsPrimitive = $codeGenerator->IsPrimitiveType($paramIDLType);
-        my $paramIsGDOMType = IsGDOMClassType($paramIDLType);
-        if (!$paramTypeIsPrimitive) {
+        my $paramTypeIsPointer = !$codeGenerator->IsNonPointerType($paramIDLType);
+        if ($paramTypeIsPointer) {
             $gReturnMacro = GetGReturnMacro($paramName, $paramIDLType, $returnType, $functionName);
             push(@cBody, $gReturnMacro);
         }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to