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

Unified Diff: tools/dom/templates/html/impl/impl_SVGElement.darttemplate

Issue 1832713002: Optimize dartium dart:html bindings so real world application performance is acceptable. Improves d… (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: update cached patches Created 4 years, 9 months 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: tools/dom/templates/html/impl/impl_SVGElement.darttemplate
diff --git a/tools/dom/templates/html/impl/impl_SVGElement.darttemplate b/tools/dom/templates/html/impl/impl_SVGElement.darttemplate
index b41f7eca2eb7beb0256dd4c7e7b8c02972bcb6f5..fec3eb6a87d8615f8e9a705f41ef1cd3c442c979 100644
--- a/tools/dom/templates/html/impl/impl_SVGElement.darttemplate
+++ b/tools/dom/templates/html/impl/impl_SVGElement.darttemplate
@@ -146,18 +146,8 @@ $(ANNOTATIONS)$(NATIVESPEC)$(CLASS_MODIFIERS)class $CLASSNAME$EXTENDS$IMPLEMENTS
$if JSINTEROP
set _svgClassName(AnimatedString value) =>
- _blink.BlinkSVGElement.instance.className_Setter_(unwrap_jso(this), unwrap_jso(value));
+ _blink.BlinkSVGElement.instance.className_Setter_(this, value);
- String get className => _svgClassName.baseVal;
-
- // Unbelievable hack. We can't create an SvgAnimatedString, but we can get
- // the existing one and change its baseVal. Then we call the blink setter directly
- // TODO(alanknight): Handle suppressing the SVGAnimated<*> better
- set className(String s) {
- var oldClass = _svgClassName;
- oldClass.baseVal = s;
- _svgClassName = oldClass;
- }
$endif
$!MEMBERS
}

Powered by Google App Engine
This is Rietveld 408576698