Title: [149572] trunk/Source/WebCore
Revision
149572
Author
[email protected]
Date
2013-05-04 18:53:46 -0700 (Sat, 04 May 2013)

Log Message

Get rid of special cases in AddIncludesForTypeInImpl
https://bugs.webkit.org/show_bug.cgi?id=115602

Reviewed by Benjamin Poulain.

Get rid of the special cases in AddIncludesForTypeInImpl subroutine in the
JS bindings generator. Those are no longer needed and special cases should
be avoided in the generator.

No new tests, no behavior change.

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

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (149571 => 149572)


--- trunk/Source/WebCore/ChangeLog	2013-05-05 01:47:47 UTC (rev 149571)
+++ trunk/Source/WebCore/ChangeLog	2013-05-05 01:53:46 UTC (rev 149572)
@@ -1,5 +1,21 @@
 2013-05-04  Christophe Dumez  <[email protected]>
 
+        Get rid of special cases in AddIncludesForTypeInImpl
+        https://bugs.webkit.org/show_bug.cgi?id=115602
+
+        Reviewed by Benjamin Poulain.
+
+        Get rid of the special cases in AddIncludesForTypeInImpl subroutine in the
+        JS bindings generator. Those are no longer needed and special cases should
+        be avoided in the generator.
+
+        No new tests, no behavior change.
+
+        * bindings/scripts/CodeGeneratorJS.pm:
+        (AddIncludesForTypeInImpl):
+
+2013-05-04  Christophe Dumez  <[email protected]>
+
         Get rid of AddIncludesForSVGAnimatedType in bindings generator
         https://bugs.webkit.org/show_bug.cgi?id=115603
 

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm (149571 => 149572)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2013-05-05 01:47:47 UTC (rev 149571)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2013-05-05 01:53:46 UTC (rev 149572)
@@ -192,25 +192,6 @@
     my $isCallback = @_ ? shift : 0;
     
     AddIncludesForType($type, $isCallback, \%implIncludes);
-    
-    # additional includes (things needed to compile the bindings but not the header)
-    if ($type eq "CanvasRenderingContext2D") {
-        $implIncludes{"CanvasGradient.h"} = 1;
-        $implIncludes{"CanvasPattern.h"} = 1;
-        $implIncludes{"CanvasStyle.h"} = 1;
-    }
-
-    if ($type eq "CanvasGradient" or $type eq "XPathNSResolver" or $type eq "MessagePort") {
-        $implIncludes{"<wtf/text/WTFString.h>"} = 1;
-    }
-
-    if ($type eq "Document") {
-        $implIncludes{"NodeFilter.h"} = 1;
-    }
-
-    if ($type eq "MediaQueryListListener") {
-        $implIncludes{"MediaQueryListListener.h"} = 1;
-    }
 }
 
 sub AddIncludesForTypeInHeader
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to