Remove support for the externalResourcesRequired attribute.

Source/core/svg/SVGImageLoader.cpp

Index: Source/core/svg/SVGImageLoader.cpp
diff --git a/Source/core/svg/SVGImageLoader.cpp b/Source/core/svg/SVGImageLoader.cpp
index e889c61b149dc3a2a1bf8dd66c0644e71f12aef9..bced77a862cabf92e580875567b1c5a1f58d300c 100644
--- a/Source/core/svg/SVGImageLoader.cpp
+++ b/Source/core/svg/SVGImageLoader.cpp
@@ -39,11 +39,6 @@ void SVGImageLoader::dispatchLoadEvent()
 {
     if (image()->errorOccurred())
         element()->dispatchEvent(Event::create(EventTypeNames::error));
-    else {
-        SVGImageElement* imageElement = toSVGImageElement(element());
-        if (imageElement->externalResourcesRequiredBaseValue())
-            imageElement->sendSVGLoadEventIfPossible(true);

[Stephen Chennney, 2013/11/06 16:45:43]

Don't we always need to dispatch a load event now, or at least modify the
sendSVGLoadEventIfPossible method? If not, do we even need
sendSVGLoadEventIfPossible?

[Erik Dahlström (inactive), 2013/11/07 09:38:04]

My preference would be to dispatch the image load event here, possibly also
triggering load on the parent outermost svg element (if all resources have
loaded). It's not clear to me why dispatchLoadEvent should dispatch an error
event here, shouldn't that be done via ImageLoader::dispatchPendingErrorEvent?

Again, I opted for not changing the code if possible in the first iteration,
just assuming that eRR returns false everywhere and seeing what falls out.

-    }
 }
 
 String SVGImageLoader::sourceURI(const AtomicString& attribute) const