Title: [131673] trunk/Source/WebCore
Revision
131673
Author
[email protected]
Date
2012-10-17 17:03:57 -0700 (Wed, 17 Oct 2012)

Log Message

[V8] Don't generate code that declares visitDOMWrapper if we're not going to generate the implementation
https://bugs.webkit.org/show_bug.cgi?id=99653

Reviewed by Eric Seidel.

* bindings/scripts/CodeGeneratorV8.pm:
(GenerateHeader):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (131672 => 131673)


--- trunk/Source/WebCore/ChangeLog	2012-10-18 00:02:25 UTC (rev 131672)
+++ trunk/Source/WebCore/ChangeLog	2012-10-18 00:03:57 UTC (rev 131673)
@@ -1,3 +1,13 @@
+2012-10-17  Adam Barth  <[email protected]>
+
+        [V8] Don't generate code that declares visitDOMWrapper if we're not going to generate the implementation
+        https://bugs.webkit.org/show_bug.cgi?id=99653
+
+        Reviewed by Eric Seidel.
+
+        * bindings/scripts/CodeGeneratorV8.pm:
+        (GenerateHeader):
+
 2012-10-17  David Grogan  <[email protected]>
 
         IndexedDB: Destroy leveldb directory if unknown schema is detected

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm (131672 => 131673)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm	2012-10-18 00:02:25 UTC (rev 131672)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm	2012-10-18 00:03:57 UTC (rev 131673)
@@ -382,9 +382,13 @@
     }
     inline static v8::Handle<v8::Object> wrap(${nativeType}*, v8::Handle<v8::Object> creationContext = v8::Handle<v8::Object>(), v8::Isolate* = 0);
     static void derefObject(void*);
-    static void visitDOMWrapper(DOMDataStore*, void*, v8::Persistent<v8::Object>);
     static WrapperTypeInfo info;
 END
+
+    if (NeedsToVisitDOMWrapper($dataNode)) {
+        push(@headerContent, "    static void visitDOMWrapper(DOMDataStore*, void*, v8::Persistent<v8::Object>);\n");
+    }
+
     if ($dataNode->extendedAttributes->{"ActiveDOMObject"}) {
         push(@headerContent, "    static ActiveDOMObject* toActiveDOMObject(v8::Handle<v8::Object>);\n");
     }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to