[SVG] Missing image loader update on re-insert.

Source/core/svg/SVGImageElement.cpp

Index: Source/core/svg/SVGImageElement.cpp
diff --git a/Source/core/svg/SVGImageElement.cpp b/Source/core/svg/SVGImageElement.cpp
index 4d9ec89f4b8735056233f713d1c4487492f01f3a..1dc68d6a61f619afc3bfc852f2d61e31d66c1249 100644
--- a/Source/core/svg/SVGImageElement.cpp
+++ b/Source/core/svg/SVGImageElement.cpp
@@ -208,6 +208,11 @@ Node::InsertionNotificationRequest SVGImageElement::insertedInto(ContainerNode*
     if (m_needsLoaderURIUpdate) {
         m_imageLoader.updateFromElementIgnoringPreviousError();
         m_needsLoaderURIUpdate = false;
+    } else {
+        // A previous loader update may have failed to actually fetch the image if the document
+        // was inactive. In that case, force a re-update (but don't clear previous errors).
+        if (!m_imageLoader.image() || !m_imageLoader.image()->hasImage())

[pdr., 2014/03/09 00:12:41]

The !m_imageLoader.image() check looks right to me. Is this second check for
hasImage necessary?

[f(malita), 2014/03/09 01:35:20]

Not for this particular failure - it's me getting paranoid and dialing it way
back :)

Let's land without.

+            m_imageLoader.updateFromElement();
     }
 
     return InsertionDone;