Phase III Step 2: Call imageNotifyFinished() and image load event after SVG loading completes

third_party/WebKit/Source/core/svg/graphics/SVGImage.h

 /*
  * Copyright (C) 2006 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 15 matching lines @@
 
 #ifndef SVGImage_h
 #define SVGImage_h
 
 #include "core/CoreExport.h"
 #include "platform/graphics/Image.h"
 #include "platform/heap/Handle.h"
 #include "platform/weborigin/KURL.h"
 #include "third_party/skia/include/core/SkRefCnt.h"
 #include "wtf/Allocator.h"
+#include "wtf/WeakPtr.h"
 
 namespace blink {
 
 class Document;
 class Page;
 class PaintController;
 class LayoutReplaced;
 class SVGImageChromeClient;
 class SVGImageForContainer;
 
@@ skipping 103 matching lines @@
                     const FloatRect& fromRect,
                     const FloatRect& toRect,
                     RespectImageOrientationEnum,
                     ImageClampingMode,
                     const KURL&);
 
   void stopAnimation();
   void scheduleTimelineRewind();
   void flushPendingTimelineRewind();
 
+  WTF::WeakPtr<SVGImage> asWeakPtr() {
+    return m_weakPtrFactory.createWeakPtr();
+  }
+
+  class SVGImageLocalFrameClient;
+
   Persistent<SVGImageChromeClient> m_chromeClient;
   Persistent<Page> m_page;
   std::unique_ptr<PaintController> m_paintController;
 
   // When an SVG image has no intrinsic size, the size depends on the default
   // object size, which in turn depends on the container. One SVGImage may
   // belong to multiple containers so the final image size can't be known in
   // SVGImage. SVGImageForContainer carries the final image size, also called
   // the "concrete object size". For more, see: SVGImageForContainer.h
   IntSize m_intrinsicSize;
   bool m_hasPendingTimelineRewind;
+
+  enum AsyncLoadCompletionStatus {

[fs, 2017/03/15 14:22:17]

Why Async...? That's just one possible outcome.

[hiroshige, 2017/05/04 22:50:50]

Done. Renamed to LoadState (and renamed some of enum value names).

+    DataChangedNotStarted,
+    InDataChanged,
+    AfterDataChanged,
+    LoadCompletedWithinDataChanged,
+    LoadCompletedAsynchronously,
+  };
+
+  AsyncLoadCompletionStatus m_loadStatus = DataChangedNotStarted;
+
+  WTF::WeakPtrFactory<SVGImage> m_weakPtrFactory;
 };
 
 DEFINE_IMAGE_TYPE_CASTS(SVGImage);
 
 class ImageObserverDisabler {
   STACK_ALLOCATED();
   WTF_MAKE_NONCOPYABLE(ImageObserverDisabler);
 
  public:
   ImageObserverDisabler(Image* image) : m_image(image) {
     m_image->setImageObserverDisabled(true);
   }
 
   ~ImageObserverDisabler() { m_image->setImageObserverDisabled(false); }
 
  private:
   Image* m_image;
 };
 
 }  // namespace blink
 
 #endif  // SVGImage_h