Refactor image handling in Sky

sky/engine/core/loader/ImageLoader.idl

Index: sky/engine/core/loader/ImageLoader.idl
diff --git a/sky/engine/core/painting/Image.idl b/sky/engine/core/loader/ImageLoader.idl
similarity index 55%
copy from sky/engine/core/painting/Image.idl
copy to sky/engine/core/loader/ImageLoader.idl
index 3d24b9aa40a1a3abc7218af9ebcb579e006686b1..c06b19c87c0b61d889fbc3e7101b215eb29718a7 100644
--- a/sky/engine/core/painting/Image.idl
+++ b/sky/engine/core/loader/ImageLoader.idl
@@ -4,9 +4,7 @@
 
 [
   Constructor(),
-  ImplementedAs=CanvasImage,
-] interface Image {
-    readonly attribute long width;
-    readonly attribute long height;
-    attribute DOMString src;
+  ImplementedAs=CanvasImageLoader,
+] interface ImageLoader {
+    void load(DOMString src, ImageLoaderCallback callback);

[abarth-chromium, 2015/06/01 22:57:09]

Should ImageLoader support multiple in-flight image loads?  The interface looks
like it does but the implementation does not...

Maybe pass the |src| and the |callback| as constructor parameters?  That way it
is clear that each instance can handle only one load.

[jackson, 2015/06/01 23:10:26]

Acknowledged.

 };