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

third_party/WebKit/Source/platform/graphics/Image.h

 /*
  * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2004, 2005, 2006 Apple Computer, 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 83 matching lines @@
 
   virtual bool UsesContainerSize() const { return false; }
   virtual bool HasRelativeSize() const { return false; }
 
   virtual IntSize Size() const = 0;
   IntRect Rect() const { return IntRect(IntPoint(), Size()); }
   int width() const { return Size().Width(); }
   int height() const { return Size().Height(); }
   virtual bool GetHotSpot(IntPoint&) const { return false; }
 
-  enum SizeAvailability { kSizeAvailable, kSizeUnavailable };
+  enum SizeAvailability {

[fs, 2017/05/05 10:52:42]

uNit: Maybe order these in a sort of "lifecycle order" (Unavailable >
AvailableAndLoadingAsync > Available)

[hiroshige, 2017/05/08 17:22:07]

Done.

+    kSizeAvailable,
+    kSizeUnavailable,
+    kSizeAvailableAndLoadingAsynchronously
+  };
+
+  // If setData() returns |SizeAvailableAndLoadingAsynchronously|:
+  //   Image loading is continuing asynchronously
+  //   (only when |this| is SVGImage and |allDataReceived| is true), and

[fs, 2017/05/05 10:52:42]

Nit: The Blink Renaming means naming here (and in other comments, seems
unnecessary to flag all) ought to be updated.

[hiroshige, 2017/05/08 17:22:06]

Done.

+  //   ImageResourceObserver::loadCompleted() is called when finished.
+  // Otherwise:
+  //   Image loading is completed synchronously.
+  //   ImageResourceObserver::loadCompleted() is not called.
   virtual SizeAvailability SetData(PassRefPtr<SharedBuffer> data,
                                    bool all_data_received);
   virtual SizeAvailability DataChanged(bool /*allDataReceived*/) {
     return kSizeUnavailable;
   }
 
   virtual String FilenameExtension() const {
     return String();
   }  // null string if unknown
 
@@ skipping 104 matching lines @@
   bool image_observer_disabled_;
 };
 
 #define DEFINE_IMAGE_TYPE_CASTS(typeName)                          \
   DEFINE_TYPE_CASTS(typeName, Image, image, image->Is##typeName(), \
                     image.Is##typeName())
 
 }  // namespace blink
 
 #endif