Title: [135387] trunk
Revision
135387
Author
hara...@chromium.org
Date
2012-11-21 05:00:55 -0800 (Wed, 21 Nov 2012)

Log Message

Unreviewed, rolling out r135340.
http://trac.webkit.org/changeset/135340
https://bugs.webkit.org/show_bug.cgi?id=102492

It broke Safari's internal build

Source/WebCore:

* bindings/scripts/CodeGenerator.pm:
(ProcessDocument):
(FileNamePrefix):
* bindings/scripts/CodeGeneratorCPP.pm:
(new):
(GenerateInterface):
(WriteData):
* bindings/scripts/CodeGeneratorGObject.pm:
(new):
(WriteData):
(GenerateInterface):
* bindings/scripts/CodeGeneratorJS.pm:
(new):
(GenerateInterface):
(WriteData):
* bindings/scripts/CodeGeneratorObjC.pm:
(new):
(GenerateInterface):
(WriteData):
* bindings/scripts/CodeGeneratorV8.pm:
(new):
(GenerateInterface):
(WriteData):

Tools:

* WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm:
(new):
(GenerateModule):
(GenerateInterface):
(finish):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (135386 => 135387)


--- trunk/Source/WebCore/ChangeLog	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Source/WebCore/ChangeLog	2012-11-21 13:00:55 UTC (rev 135387)
@@ -1,3 +1,35 @@
+2012-11-21  Kentaro Hara  <hara...@chromium.org>
+
+        Unreviewed, rolling out r135340.
+        http://trac.webkit.org/changeset/135340
+        https://bugs.webkit.org/show_bug.cgi?id=102492
+
+        It broke Safari's internal build
+
+        * bindings/scripts/CodeGenerator.pm:
+        (ProcessDocument):
+        (FileNamePrefix):
+        * bindings/scripts/CodeGeneratorCPP.pm:
+        (new):
+        (GenerateInterface):
+        (WriteData):
+        * bindings/scripts/CodeGeneratorGObject.pm:
+        (new):
+        (WriteData):
+        (GenerateInterface):
+        * bindings/scripts/CodeGeneratorJS.pm:
+        (new):
+        (GenerateInterface):
+        (WriteData):
+        * bindings/scripts/CodeGeneratorObjC.pm:
+        (new):
+        (GenerateInterface):
+        (WriteData):
+        * bindings/scripts/CodeGeneratorV8.pm:
+        (new):
+        (GenerateInterface):
+        (WriteData):
+
 2012-11-21  Pavel Feldman  <pfeld...@chromium.org>
 
         Web Inspector: reparenting view does not detach from the previous parent.

Modified: trunk/Source/WebCore/bindings/scripts/CodeGenerator.pm (135386 => 135387)


--- trunk/Source/WebCore/bindings/scripts/CodeGenerator.pm	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Source/WebCore/bindings/scripts/CodeGenerator.pm	2012-11-21 13:00:55 UTC (rev 135387)
@@ -124,7 +124,7 @@
     require $ifaceName . ".pm";
 
     # Dynamically load external code generation perl module
-    $codeGenerator = $ifaceName->new($object, $useLayerOnTop, $preprocessor, $writeDependencies, $verbose, $targetIdlFilePath);
+    $codeGenerator = $ifaceName->new($object, $useOutputDir, $useOutputHeadersDir, $useLayerOnTop, $preprocessor, $writeDependencies, $verbose, $targetIdlFilePath);
     unless (defined($codeGenerator)) {
         my $interfaces = $useDocument->interfaces;
         foreach my $interface (@$interfaces) {
@@ -137,7 +137,6 @@
     foreach my $interface (@$interfaces) {
         print "Generating $useGenerator bindings code for IDL interface \"" . $interface->name . "\"...\n" if $verbose;
         $codeGenerator->GenerateInterface($interface, $defines);
-        $codeGenerator->WriteData($interface, $useOutputDir, $useOutputHeadersDir);
     }
 }
 
@@ -149,7 +148,7 @@
     require $ifaceName . ".pm";
 
     # Dynamically load external code generation perl module
-    $codeGenerator = $ifaceName->new($object, $useLayerOnTop, $preprocessor, $writeDependencies, $verbose);
+    $codeGenerator = $ifaceName->new($object, $useOutputDir, $useOutputHeadersDir, $useLayerOnTop, $preprocessor, $writeDependencies, $verbose);
     return $codeGenerator->FileNamePrefix();
 }
 

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorCPP.pm (135386 => 135387)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorCPP.pm	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorCPP.pm	2012-11-21 13:00:55 UTC (rev 135387)
@@ -28,6 +28,7 @@
 use constant FileNamePrefix => "WebDOM";
 
 # Global Variables
+my $outputDir = "";
 
 my @headerContentHeader = ();
 my @headerContent = ();
@@ -95,6 +96,8 @@
     my $reference = { };
 
     $codeGenerator = shift;
+    $outputDir = shift;
+    shift; # $outputHeadersDir
     shift; # $useLayerOnTop
     shift; # $preprocessor
     shift; # $writeDependencies
@@ -113,8 +116,12 @@
     my $className = GetClassName($name);
     my $parentClassName = "WebDOM" . GetParentImplClassName($interface);
 
+    # Start actual generation.
     $object->GenerateHeader($interface);
     $object->GenerateImplementation($interface);
+
+    # Write changes.
+    $object->WriteData(FileNamePrefix . $name);
 }
 
 sub GetClassName
@@ -928,17 +935,15 @@
     push(@implContent, "\n#endif // ${conditionalString}\n") if $conditionalString;
 }
 
+# Internal helper
 sub WriteData
 {
     my $object = shift;
-    my $dataNode = shift;
-    my $outputDir = shift;
+    my $name = shift;
 
     # Open files for writing...
-    my $name = $dataNode->name;
-    my $prefix = FileNamePrefix;
-    my $headerFileName = "$outputDir/$prefix$name.h";
-    my $implFileName = "$outputDir/$prefix$name.cpp";
+    my $headerFileName = "$outputDir/" . $name . ".h";
+    my $implFileName = "$outputDir/" . $name . ".cpp";
 
     # Update a .h file if the contents are changed.
     my $contents = join "", @headerContentHeader;

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm (135386 => 135387)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorGObject.pm	2012-11-21 13:00:55 UTC (rev 135387)
@@ -45,6 +45,8 @@
     my $reference = { };
 
     $codeGenerator = shift;
+    $outputDir = shift;
+    mkdir $outputDir;
 
     bless($reference, $object);
 }
@@ -1314,11 +1316,10 @@
     $object->GenerateEndHeader();
 }
 
+# Internal helper
 sub WriteData {
     my $object = shift;
     my $interface = shift;
-    my $outputDir = shift;
-    mkdir $outputDir;
 
     # Write a private header.
     my $interfaceName = $interface->name;
@@ -1432,6 +1433,7 @@
     $className = GetClassName($interface->name);
 
     $object->Generate($interface);
+    $object->WriteData($interface);
 }
 
 1;

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm (135386 => 135387)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2012-11-21 13:00:55 UTC (rev 135387)
@@ -32,6 +32,7 @@
 
 my $codeGenerator;
 
+my $outputDir = "";
 my $writeDependencies = 0;
 
 my @headerContentHeader = ();
@@ -76,6 +77,8 @@
     my $reference = { };
 
     $codeGenerator = shift;
+    $outputDir = shift;
+    shift; # $outputHeadersDir
     shift; # $useLayerOnTop
     shift; # $preprocessor
     $writeDependencies = shift;
@@ -105,6 +108,8 @@
         $object->GenerateHeader($interface);
         $object->GenerateImplementation($interface);
     }
+
+    $object->WriteData($interface);
 }
 
 sub GenerateAttributeEventListenerCall
@@ -3530,11 +3535,11 @@
     return $hash;
 }
 
+# Internal helper
 sub WriteData
 {
     my $object = shift;
     my $interface = shift;
-    my $outputDir = shift;
 
     my $name = $interface->name;
     my $prefix = FileNamePrefix;

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorObjC.pm (135386 => 135387)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorObjC.pm	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorObjC.pm	2012-11-21 13:00:55 UTC (rev 135387)
@@ -29,6 +29,7 @@
 use constant FileNamePrefix => "DOM";
 
 # Global Variables
+my $outputDir = "";
 my $writeDependencies = 0;
 my %publicInterfaces = ();
 my $newPublicClass = 0;
@@ -194,6 +195,8 @@
     my $reference = { };
 
     $codeGenerator = shift;
+    $outputDir = shift;
+    shift; # $outputHeadersDir
     shift; # $useLayerOnTop
     shift; # $preprocessor
     $writeDependencies = shift;
@@ -283,6 +286,9 @@
     $object->GenerateHeader($interface);
     $object->GenerateImplementation($interface) unless $noImpl;
 
+    # Write changes.
+    $object->WriteData(FileNamePrefix . $name);
+
     # Check for missing public API
     if (keys %publicInterfaces > 0) {
         my $missing = join("\n", keys %publicInterfaces);
@@ -1746,17 +1752,14 @@
 sub WriteData
 {
     my $object = shift;
-    my $dataNode = shift;
-    my $outputDir = shift;
+    my $name = shift;
 
     # Open files for writing...
-    my $name = $dataNode->name;
-    my $prefix = FileNamePrefix;
-    my $headerFileName = "$outputDir/$prefix$name.h";
-    my $privateHeaderFileName = "$outputDir/$prefix${name}Private.h";
-    my $implFileName = "$outputDir/$prefix$name.mm";
-    my $internalHeaderFileName = "$outputDir/$prefix${name}Internal.h";
-    my $depsFileName = "$outputDir/$prefix$name.dep";
+    my $headerFileName = "$outputDir/" . $name . ".h";
+    my $privateHeaderFileName = "$outputDir/" . $name . "Private.h";
+    my $implFileName = "$outputDir/" . $name . ".mm";
+    my $internalHeaderFileName = "$outputDir/" . $name . "Internal.h";
+    my $depsFileName = "$outputDir/" . $name . ".dep";
 
     # Write public header.
     my $contents = join "", @headerContentHeader;

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm (135386 => 135387)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm	2012-11-21 13:00:55 UTC (rev 135387)
@@ -32,6 +32,8 @@
 
 my $codeGenerator;
 
+my $outputDir = "";
+my $outputHeadersDir = "";
 
 my @headerContent = ();
 my @implContentHeader = ();
@@ -70,6 +72,8 @@
     my $reference = { };
 
     $codeGenerator = shift;
+    $outputDir = shift;
+    $outputHeadersDir = shift;
 
     bless($reference, $object);
     return $reference;
@@ -88,6 +92,8 @@
         $object->GenerateHeader($interface);
         $object->GenerateImplementation($interface);
     }
+
+    $object->WriteData($interface);
 }
 
 sub AddToImplIncludes
@@ -4133,12 +4139,11 @@
     return "toV8($value$getCreationContextArg$getIsolateArg)";
 }
 
+# Internal helper
 sub WriteData
 {
     my $object = shift;
     my $interface = shift;
-    my $outputDir = shift;
-    my $outputHeadersDir = shift;
 
     my $name = $interface->name;
     my $prefix = FileNamePrefix;

Modified: trunk/Tools/ChangeLog (135386 => 135387)


--- trunk/Tools/ChangeLog	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Tools/ChangeLog	2012-11-21 13:00:55 UTC (rev 135387)
@@ -1,3 +1,17 @@
+2012-11-21  Kentaro Hara  <hara...@chromium.org>
+
+        Unreviewed, rolling out r135340.
+        http://trac.webkit.org/changeset/135340
+        https://bugs.webkit.org/show_bug.cgi?id=102492
+
+        It broke Safari's internal build
+
+        * WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm:
+        (new):
+        (GenerateModule):
+        (GenerateInterface):
+        (finish):
+
 2012-11-21  Yael Aharon  <yael.aha...@intel.com>, Gyuyoung Kim <gyuyoung....@samsung.com>
 
         [EFL] Turn on WTF_USE_TILED_BACKING_STORE by default

Modified: trunk/Tools/WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm (135386 => 135387)


--- trunk/Tools/WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm	2012-11-21 12:37:04 UTC (rev 135386)
+++ trunk/Tools/WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm	2012-11-21 13:00:55 UTC (rev 135387)
@@ -30,10 +30,11 @@
 
 sub new
 {
-    my ($class, $codeGenerator, $layerOnTop, $preprocessor, $writeDependencies, $verbose, $idlFilePath) = @_;
+    my ($class, $codeGenerator, $outputDir, $outputHeaderDir, $layerOnTop, $preprocessor, $writeDependencies, $verbose, $idlFilePath) = @_;
 
     my $reference = {
         codeGenerator => $codeGenerator,
+        outputDir => $outputDir,
         idlFilePath => $idlFilePath,
     };
 
@@ -41,21 +42,25 @@
     return $reference;
 }
 
-sub GenerateInterface
+sub GenerateModule
 {
 }
 
-sub WriteData
+sub GenerateInterface
 {
-    my ($self, $interface, $outputDir) = @_;
+    my ($self, $interface, $defines) = @_;
 
     foreach my $file ($self->_generateHeaderFile($interface), $self->_generateImplementationFile($interface)) {
-        open(FILE, ">", File::Spec->catfile($outputDir, $$file{name})) or die "Failed to open $$file{name} for writing: $!";
+        open(FILE, ">", File::Spec->catfile($$self{outputDir}, $$file{name})) or die "Failed to open $$file{name} for writing: $!";
         print FILE @{$$file{contents}};
         close(FILE) or die "Failed to close $$file{name} after writing: $!";
     }
 }
 
+sub finish
+{
+}
+
 sub _className
 {
     my ($idlType) = @_;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to