Title: [141502] trunk/Source/WebCore
- Revision
- 141502
- Author
- hara...@chromium.org
- Date
- 2013-01-31 16:25:43 -0800 (Thu, 31 Jan 2013)
Log Message
[V8] Simplify CodeGeneratorV8.pm by using InheritsExtendedAttribute("EventTarget")
https://bugs.webkit.org/show_bug.cgi?id=108441
Reviewed by Adam Barth.
A complicated condition in GetInternalFields() can be simplified
by using InheritsExtendedAttribute("EventTarget").
No tests. No change in behavior.
* bindings/scripts/CodeGeneratorV8.pm:
(GetInternalFields):
* dom/EventTarget.idl: Added [EventTarget] which should have been added.
* svg/SVGElementInstance.idl: Ditto.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (141501 => 141502)
--- trunk/Source/WebCore/ChangeLog 2013-01-31 23:59:29 UTC (rev 141501)
+++ trunk/Source/WebCore/ChangeLog 2013-02-01 00:25:43 UTC (rev 141502)
@@ -1,3 +1,20 @@
+2013-01-31 Kentaro Hara <hara...@chromium.org>
+
+ [V8] Simplify CodeGeneratorV8.pm by using InheritsExtendedAttribute("EventTarget")
+ https://bugs.webkit.org/show_bug.cgi?id=108441
+
+ Reviewed by Adam Barth.
+
+ A complicated condition in GetInternalFields() can be simplified
+ by using InheritsExtendedAttribute("EventTarget").
+
+ No tests. No change in behavior.
+
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GetInternalFields):
+ * dom/EventTarget.idl: Added [EventTarget] which should have been added.
+ * svg/SVGElementInstance.idl: Ditto.
+
2013-01-31 Youenn Fablet <youe...@gmail.com>
Quadratic and bezier curves with coincident endpoints rendered incorrectly
Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm (141501 => 141502)
--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm 2013-01-31 23:59:29 UTC (rev 141501)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm 2013-02-01 00:25:43 UTC (rev 141502)
@@ -616,18 +616,11 @@
my $interface = shift;
my @customInternalFields = ();
- # We can't ask whether a parent type has a given extendedAttribute,
- # 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->InheritsInterface($interface, "Node")
- && ($interface->extendedAttributes->{"EventTarget"}
- || $interface->extendedAttributes->{"IsWorkerContext"}
- || $codeGenerator->InheritsInterface($interface, "AbstractWorker")
- || $codeGenerator->InheritsInterface($interface, "EventTarget"))) {
+ if (!$codeGenerator->InheritsInterface($interface, "Node") &&
+ $codeGenerator->InheritsExtendedAttribute($interface, "EventTarget")) {
push(@customInternalFields, "eventListenerCacheIndex");
}
-
return @customInternalFields;
}
Modified: trunk/Source/WebCore/dom/EventTarget.idl (141501 => 141502)
--- trunk/Source/WebCore/dom/EventTarget.idl 2013-01-31 23:59:29 UTC (rev 141501)
+++ trunk/Source/WebCore/dom/EventTarget.idl 2013-02-01 00:25:43 UTC (rev 141502)
@@ -25,7 +25,8 @@
OmitConstructor,
CustomToJSObject,
JSCustomToNativeObject,
- V8NoWrapperCache
+ V8NoWrapperCache,
+ EventTarget
] interface EventTarget {
[ObjCLegacyUnnamedParameters] void addEventListener(in DOMString type,
in EventListener listener,
Modified: trunk/Source/WebCore/svg/SVGElementInstance.idl (141501 => 141502)
--- trunk/Source/WebCore/svg/SVGElementInstance.idl 2013-01-31 23:59:29 UTC (rev 141501)
+++ trunk/Source/WebCore/svg/SVGElementInstance.idl 2013-02-01 00:25:43 UTC (rev 141502)
@@ -28,7 +28,8 @@
Conditional=SVG,
JSCustomMarkFunction,
JSGenerateToNativeObject,
- JSGenerateToJSObject
+ JSGenerateToJSObject,
+ EventTarget
] interface SVGElementInstance
#if defined(LANGUAGE_OBJECTIVE_C) && LANGUAGE_OBJECTIVE_C
: Object, EventTarget
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes