Remove support for the externalResourcesRequired attribute.

Source/core/svg/SVGImageLoader.cpp

 /*
  * Copyright (C) 2005, 2005 Alexander Kellett <lypanov@kde.org>
  * Copyright (C) 2008 Rob Buis <buis@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 21 matching lines @@
 
 SVGImageLoader::SVGImageLoader(SVGImageElement* node)
     : ImageLoader(node)
 {
 }
 
 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
 {
     KURL base = element()->baseURI();
     if (base.isValid())
         return KURL(base, stripLeadingAndTrailingHTMLSpaces(attribute)).string();
     return element()->document().completeURL(stripLeadingAndTrailingHTMLSpaces(attribute));
 }
 
 }