Title: [140884] trunk/Source/WebCore
Revision
140884
Author
[email protected]
Date
2013-01-25 17:05:54 -0800 (Fri, 25 Jan 2013)

Log Message

Rename CodeGenerator::IsSubType() to CodeGenerator::InheritsInterface()
https://bugs.webkit.org/show_bug.cgi?id=107874

Reviewed by Adam Barth.

No tests. No change in behavior.

* bindings/scripts/CodeGenerator.pm:
(InheritsInterface):
* bindings/scripts/CodeGeneratorJS.pm:
(GenerateHeader):
(GenerateImplementation):
* bindings/scripts/CodeGeneratorV8.pm:
(GenerateHeader):
(GetInternalFields):
(GenerateNormalAttrGetter):
(GenerateNormalAttrSetter):
(GenerateFunctionCallback):
(GenerateImplementationIndexer):
(GenerateToV8Converters):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (140883 => 140884)


--- trunk/Source/WebCore/ChangeLog	2013-01-26 00:57:12 UTC (rev 140883)
+++ trunk/Source/WebCore/ChangeLog	2013-01-26 01:05:54 UTC (rev 140884)
@@ -1,5 +1,28 @@
 2013-01-25  Kentaro Hara  <[email protected]>
 
+        Rename CodeGenerator::IsSubType() to CodeGenerator::InheritsInterface()
+        https://bugs.webkit.org/show_bug.cgi?id=107874
+
+        Reviewed by Adam Barth.
+
+        No tests. No change in behavior.
+
+        * bindings/scripts/CodeGenerator.pm:
+        (InheritsInterface):
+        * bindings/scripts/CodeGeneratorJS.pm:
+        (GenerateHeader):
+        (GenerateImplementation):
+        * bindings/scripts/CodeGeneratorV8.pm:
+        (GenerateHeader):
+        (GetInternalFields):
+        (GenerateNormalAttrGetter):
+        (GenerateNormalAttrSetter):
+        (GenerateFunctionCallback):
+        (GenerateImplementationIndexer):
+        (GenerateToV8Converters):
+
+2013-01-25  Kentaro Hara  <[email protected]>
+
         [V8] Pass an Isolate to GetRawTemplate()
         https://bugs.webkit.org/show_bug.cgi?id=107927
 

Modified: trunk/Source/WebCore/bindings/scripts/CodeGenerator.pm (140883 => 140884)


--- trunk/Source/WebCore/bindings/scripts/CodeGenerator.pm	2013-01-26 00:57:12 UTC (rev 140883)
+++ trunk/Source/WebCore/bindings/scripts/CodeGenerator.pm	2013-01-26 01:05:54 UTC (rev 140884)
@@ -732,8 +732,7 @@
     return $interfaceName ? $interfaceName : $interface->name;
 }
 
-# FIXME: Rename to InheritsInterface
-sub IsSubType
+sub InheritsInterface
 {
     my $object = shift;
     my $interface = shift;

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm (140883 => 140884)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2013-01-26 00:57:12 UTC (rev 140883)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2013-01-26 01:05:54 UTC (rev 140884)
@@ -1040,7 +1040,7 @@
         GetCustomIsReachable($interface) ||
         $interface->extendedAttributes->{"JSCustomFinalize"} ||
         $interface->extendedAttributes->{"ActiveDOMObject"}) {
-        if ($interfaceName ne "Node" && $codeGenerator->IsSubType($interface, "Node")) {
+        if ($interfaceName ne "Node" && $codeGenerator->InheritsInterface($interface, "Node")) {
             $headerIncludes{"JSNode.h"} = 1;
             push(@headerContent, "class JS${interfaceName}Owner : public JSNodeOwner {\n");
         } else {
@@ -1413,7 +1413,7 @@
     my $hasParent = $hasLegacyParent || $hasRealParent;
     my $parentClassName = GetParentClassName($interface);
     my $visibleInterfaceName = $codeGenerator->GetVisibleInterfaceName($interface);
-    my $eventTarget = $interface->extendedAttributes->{"EventTarget"} || ($codeGenerator->IsSubType($interface, "EventTarget") && $interface->name ne "EventTarget");
+    my $eventTarget = $interface->extendedAttributes->{"EventTarget"} || ($codeGenerator->InheritsInterface($interface, "EventTarget") && $interface->name ne "EventTarget");
     my $needsMarkChildren = $interface->extendedAttributes->{"JSCustomMarkFunction"} || $interface->extendedAttributes->{"EventTarget"} || $interface->name eq "EventTarget";
 
     # - Add default header template
@@ -2492,7 +2492,7 @@
             push(@implContent, "    if (js${interfaceName}->impl()->hasPendingActivity())\n");
             push(@implContent, "        return true;\n");
         }
-        if ($codeGenerator->IsSubType($interface, "Node")) {
+        if ($codeGenerator->InheritsInterface($interface, "Node")) {
             push(@implContent, "    if (JSNodeOwner::isReachableFromOpaqueRoots(handle, 0, visitor))\n");
             push(@implContent, "        return true;\n");
         }

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm (140883 => 140884)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm	2013-01-26 00:57:12 UTC (rev 140883)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm	2013-01-26 01:05:54 UTC (rev 140884)
@@ -272,7 +272,7 @@
     $codeGenerator->LinkOverloadedFunctions($interface);
 
     # Ensure the IsDOMNodeType function is in sync.
-    die("IsDOMNodeType is out of date with respect to $interfaceName") if IsDOMNodeType($interfaceName) != $codeGenerator->IsSubType($interface, "Node");
+    die("IsDOMNodeType is out of date with respect to $interfaceName") if IsDOMNodeType($interfaceName) != $codeGenerator->InheritsInterface($interface, "Node");
 
     my $hasDependentLifetime = $interface->extendedAttributes->{"V8DependentLifetime"} || $interface->extendedAttributes->{"ActiveDOMObject"}
          || GetGenerateIsReachable($interface) || $v8InterfaceName =~ /SVG/;
@@ -621,11 +621,11 @@
     # so special-case AbstractWorker and WorkerContext to include all sub-types.
     # Event listeners on DOM nodes are explicitly supported in the GC controller.
     # FIXME: Simplify this when all EventTargets are subtypes of EventTarget.
-    if (!$codeGenerator->IsSubType($interface, "Node")
+    if (!$codeGenerator->InheritsInterface($interface, "Node")
         && ($interface->extendedAttributes->{"EventTarget"}
             || $interface->extendedAttributes->{"IsWorkerContext"}
-            || $codeGenerator->IsSubType($interface, "AbstractWorker")
-            || $codeGenerator->IsSubType($interface, "EventTarget"))) {
+            || $codeGenerator->InheritsInterface($interface, "AbstractWorker")
+            || $codeGenerator->InheritsInterface($interface, "EventTarget"))) {
         push(@customInternalFields, "eventListenerCacheIndex");
     }
 
@@ -928,7 +928,7 @@
     } else {
         my $reflect = $attribute->signature->extendedAttributes->{"Reflect"};
         my $url = ""
-        if ($getterStringUsesImp && $reflect && !$url && $codeGenerator->IsSubType($interface, "Node") && $codeGenerator->IsStringType($attrType)) {
+        if ($getterStringUsesImp && $reflect && !$url && $codeGenerator->InheritsInterface($interface, "Node") && $codeGenerator->IsStringType($attrType)) {
             # Generate super-compact call for regular attribute getter:
             my ($functionName, @arguments) = $codeGenerator->GetterExpression(\%implIncludes, $interfaceName, $attribute);
             push(@implContentDecls, "    Element* imp = V8Element::toNative(info.Holder());\n");
@@ -1018,7 +1018,7 @@
     # Special case for readonly or Replaceable attributes (with a few exceptions). This attempts to ensure that JS wrappers don't get
     # garbage-collected prematurely when their lifetime is strongly tied to their owner. We accomplish this by inserting a reference to
     # the newly created wrapper into an internal field of the holder object.
-    if ((!$codeGenerator->IsSubType($interface, "Node") && $attrName ne "self" && IsWrapperType($returnType) && (IsReadonly($attribute) || $attribute->signature->extendedAttributes->{"Replaceable"} || $attrName eq "location")
+    if ((!$codeGenerator->InheritsInterface($interface, "Node") && $attrName ne "self" && IsWrapperType($returnType) && (IsReadonly($attribute) || $attribute->signature->extendedAttributes->{"Replaceable"} || $attrName eq "location")
          && $returnType ne "EventTarget" && $returnType ne "SerializedScriptValue" && $returnType ne "DOMWindow"
          && $returnType ne "MessagePortArray"
          && $returnType !~ /SVG/ && $returnType !~ /HTML/ && !IsDOMNodeType($returnType))
@@ -1206,7 +1206,7 @@
     } else {
         my $attrType = $attribute->signature->type;
         my $reflect = $attribute->signature->extendedAttributes->{"Reflect"};
-        if ($reflect && $codeGenerator->IsSubType($interface, "Node") && $codeGenerator->IsStringType($attrType)) {
+        if ($reflect && $codeGenerator->InheritsInterface($interface, "Node") && $codeGenerator->IsStringType($attrType)) {
             # Generate super-compact call for regular attribute setter:
             my $contentAttributeName = $reflect eq "VALUE_IS_MISSING" ? lc $attrName : $reflect;
             my $namespace = $codeGenerator->NamespaceForAttributeName($interfaceName, $contentAttributeName);
@@ -1265,7 +1265,7 @@
         if ($attribute->signature->type eq "EventListener") {
             my $implSetterFunctionName = $codeGenerator->WK_ucfirst($attrName);
             AddToImplIncludes("V8AbstractEventListener.h");
-            if (!$codeGenerator->IsSubType($interface, "Node")) {
+            if (!$codeGenerator->InheritsInterface($interface, "Node")) {
                 push(@implContentDecls, "    transferHiddenDependency(info.Holder(), imp->$attrName(), value, ${v8InterfaceName}::eventListenerCacheIndex);\n");
             }
             AddToImplIncludes("V8EventListenerList.h");
@@ -1507,10 +1507,10 @@
     # but they are extremely consistent across the various interfaces that take event listeners,
     # so we can generate them as a "special case".
     if ($name eq "addEventListener") {
-        GenerateEventListenerCallback($interfaceName, !$codeGenerator->IsSubType($interface, "Node"), "add");
+        GenerateEventListenerCallback($interfaceName, !$codeGenerator->InheritsInterface($interface, "Node"), "add");
         return;
     } elsif ($name eq "removeEventListener") {
-        GenerateEventListenerCallback($interfaceName, !$codeGenerator->IsSubType($interface, "Node"), "remove");
+        GenerateEventListenerCallback($interfaceName, !$codeGenerator->InheritsInterface($interface, "Node"), "remove");
         return;
     }
 
@@ -2458,7 +2458,7 @@
         }
     }
 
-    my $hasEnumerator = !$isSpecialCase && $codeGenerator->IsSubType($interface, "Node");
+    my $hasEnumerator = !$isSpecialCase && $codeGenerator->InheritsInterface($interface, "Node");
 
     # FIXME: Find a way to not have to special-case HTMLOptionsCollection.
     if ($interfaceName eq "HTMLOptionsCollection") {
@@ -3485,7 +3485,7 @@
 
     AddToImplIncludes("Frame.h");
 
-    if ($codeGenerator->IsSubType($interface, "Document")) {
+    if ($codeGenerator->InheritsInterface($interface, "Document")) {
         push(@implContent, <<END);
     if (Frame* frame = impl->frame()) {
         if (frame->script()->initializeMainWorld()) {
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to