Bug 125940 – Refactor CodeGeneratorJS - Move attribute function creation out of getOwnPropertyName guard

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Oliver Hunt]

Refactor CodeGeneratorJS - Move attribute function creation out of getOwnPropertyName guard

[Oliver Hunt]

Created attachment 219560 [details]
Patch

[Oliver Hunt]

diff ignoring whitespace changes:
diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 5845836..c1efc95 100644
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,5 +1,18 @@
 2013-12-18  Oliver Hunt  <[e-mail]>
 
+        Refactor CodeGeneratorJS - Move attribute function creation out of getOwnPropertyName guard
+        https://bugs.webkit.org/show_bug.cgi?id=125940
+
+        Reviewed by NOBODY (OOPS!).
+
+        This is just a huge block move of code out from behind the
+        ImplementationOverridesGetOwnProperty guard.
+
+        * bindings/scripts/CodeGeneratorJS.pm:
+        (GenerateImplementation):
+
[phone]  Oliver Hunt  <[e-mail]>
+
         Simplify bindings codegen for adding getOwnPropertySlot overrides
         https://bugs.webkit.org/show_bug.cgi?id=125934
 
diff --git a/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm b/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm
index dfbb5bd..43bde48 100644
--- a/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm
+++ b/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm
@@ -1874,6 +1874,7 @@ sub GenerateImplementation
             push(@implContent, "}\n\n");
         }
 
+    }
     if ($numAttributes > 0) {
         foreach my $attribute (@{$interface->attributes}) {
             my $name = $attribute->signature->name;
@@ -2066,7 +2067,7 @@ sub GenerateImplementation
             push(@implContent, "    return JSValue::encode(${className}::getConstructor(exec->vm(), domObject->globalObject()));\n");
             push(@implContent, "}\n\n");
         }
-        }
+
 
         # Check if we have any writable attributes
         my $hasReadWriteProperties = 0;
@@ -2287,6 +2288,7 @@ sub GenerateImplementation
                     }
                 }
             }
+        }
 
         if ($interface->extendedAttributes->{"ReplaceableConstructor"}) {
             my $constructorFunctionName = "setJS" . $interfaceName . "Constructor";
@@ -2312,7 +2314,6 @@ sub GenerateImplementation
             push(@implContent, "}\n\n");
         }
     }
-    }
 
     if ($indexedGetterFunction && !$interface->extendedAttributes->{"CustomEnumerateProperty"}) {
         push(@implContent, "void ${className}::getOwnPropertyNames(JSObject* object, ExecState* exec, PropertyNameArray& propertyNames, EnumerationMode mode)\n");

[Simon Fraser (smfr)]

Comment on attachment 219560 [details]
Patch

rs=me

[Oliver Hunt]

Committed r160793: <http://trac.webkit.org/changeset/160793>