Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(326)

Unified Diff: Source/bindings/scripts/code_generator_v8.pm

Issue 112003003: [SVG] SVGLength{,List} migration to new SVG property impl. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebaselined Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/bindings/scripts/code_generator_v8.pm
diff --git a/Source/bindings/scripts/code_generator_v8.pm b/Source/bindings/scripts/code_generator_v8.pm
index 5c4a4e4a5acc3176d3432509cec33691bb9ae698..ef551ba058b054942e8ace0ea95b496d5c246e37 100644
--- a/Source/bindings/scripts/code_generator_v8.pm
+++ b/Source/bindings/scripts/code_generator_v8.pm
@@ -213,10 +213,13 @@ my %svgAttributesInHTMLHash = ("class" => 1, "id" => 1, "onabort" => 1, "onclick
"onmouseup" => 1, "onresize" => 1, "onscroll" => 1,
"onunload" => 1);
+my %svgTypeNewPropertyImplementation = (
+ "SVGLength" => 1,
+ "SVGLengthList" => 1,
+);
+
my %svgTypeNeedingTearOff = (
"SVGAngle" => "SVGPropertyTearOff<SVGAngle>",
- "SVGLength" => "SVGPropertyTearOff<SVGLength>",
- "SVGLengthList" => "SVGListPropertyTearOff<SVGLengthList>",
"SVGMatrix" => "SVGMatrixTearOff",
"SVGNumber" => "SVGPropertyTearOff<SVGNumber>",
"SVGNumberList" => "SVGListPropertyTearOff<SVGNumberList>",
@@ -6081,6 +6084,8 @@ sub IsSVGAnimatedType
{
my $type = shift;
+ return 0 if $svgTypeNewPropertyImplementation{$type};
+
return $type =~ /^SVGAnimated/;
}

Powered by Google App Engine
This is Rietveld 408576698