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

Unified Diff: Source/core/svg/SVGGElement.cpp

Issue 62083002: Remove support for the externalResourcesRequired attribute. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 years, 1 month 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/core/svg/SVGGElement.cpp
diff --git a/Source/core/svg/SVGGElement.cpp b/Source/core/svg/SVGGElement.cpp
index 20e2dd875fad6a702b0066b39097f7282c87a14a..ac444b78f2cf63ca79d38db5e97ce95663db6e29 100644
--- a/Source/core/svg/SVGGElement.cpp
+++ b/Source/core/svg/SVGGElement.cpp
@@ -31,10 +31,8 @@
namespace WebCore {
// Animated property definitions
-DEFINE_ANIMATED_BOOLEAN(SVGGElement, SVGNames::externalResourcesRequiredAttr, ExternalResourcesRequired, externalResourcesRequired)
BEGIN_REGISTER_ANIMATED_PROPERTIES(SVGGElement)
- REGISTER_LOCAL_ANIMATED_PROPERTY(externalResourcesRequired)
REGISTER_PARENT_ANIMATED_PROPERTIES(SVGGraphicsElement)
END_REGISTER_ANIMATED_PROPERTIES
@@ -54,8 +52,6 @@ PassRefPtr<SVGGElement> SVGGElement::create(const QualifiedName& tagName, Docume
bool SVGGElement::isSupportedAttribute(const QualifiedName& attrName)
Stephen Chennney 2013/11/06 16:45:43 This can just return false now. If it's a virtual
Erik Dahlström (inactive) 2013/11/07 09:38:04 Actually it was buggy before, it should have had S
{
DEFINE_STATIC_LOCAL(HashSet<QualifiedName>, supportedAttributes, ());
- if (supportedAttributes.isEmpty())
- SVGExternalResourcesRequired::addSupportedAttributes(supportedAttributes);
return supportedAttributes.contains<SVGAttributeHashTranslator>(attrName);
}

Powered by Google App Engine
This is Rietveld 408576698