Check that SVGImage is loaded in ImageNotifyFinished()

third_party/WebKit/Source/core/loader/ImageLoader.cpp

Index: third_party/WebKit/Source/core/loader/ImageLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/ImageLoader.cpp b/third_party/WebKit/Source/core/loader/ImageLoader.cpp
index d1bf60bbf24ab085b06a6ac0df22fe195118f243..bbd5ae68bb23091a77853eecb5d133b63b76857a 100644
--- a/third_party/WebKit/Source/core/loader/ImageLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/ImageLoader.cpp
@@ -555,9 +555,11 @@ void ImageLoader::ImageNotifyFinished(ImageResourceContent* resource) {
 
   UpdateLayoutObject();
 
-  if (image_ && image_->GetImage() && image_->GetImage()->IsSVGImage())
+  if (image_ && image_->GetImage() && image_->GetImage()->IsSVGImage()) {
+    ToSVGImage(image_->GetImage())->CheckLoaded();

[pdr., 2017/05/19 17:16:42]

Can you add a comment here describing why it would be dangerous for this to
fail? I'd just like to help a future engineer years from now who finds this line
and thinks it can be removed.

[hiroshige, 2017/05/19 20:51:44]

Added comments here and in CheckLoaded().
SVG people, do these comment look reasonable?

     ToSVGImage(image_->GetImage())
         ->UpdateUseCounters(GetElement()->GetDocument());
+  }
 
   if (loading_image_document_) {
     CHECK(!has_pending_load_event_);