got play-sound working

whalesong/image/private/js-impl.js

@@ -385,6 +385,56 @@ EXPORTS['video/url'] =
             );
         });
 
+// We keep a cache of loaded sounds:
+var audioCache = {};
+
+EXPORTS['play-sound'] =
+    makeClosure(
+        'play-sound',
+        1,
+        function(MACHINE) {
+            var path = checkString(MACHINE, 'play-sound', 0);
+            var fileAudio = audioCache[path];
+            if (fileAudio) {
+                // the sound was already loaded
+                finalizeClosureCall(
+                    MACHINE,
+                    fileAudio.play());
+            }
+            else {
+               // this sound has never been played before
+               PAUSE(
+                   function(restart) {
+                      fileAudio = makeFileAudio(path.toString());
+                      audioCache[path] = fileAudio;
+                      // let the audio file load before playing...
+                      fileAudio.loading = true;
+                      // (fileAudio.audio is the raw html5 Audio object)
+                      fileAudio.audio.addEventListener('canplay', function() {
+                          // ignore canplay events that follow the initial load
+                          if(fileAudio.loading) {
+                            restart(function(MACHINE) {
+                              finalizeClosureCall(
+                                  MACHINE,
+                                  fileAudio.play());
+                              });
+                            fileAudio.loading = false; // we're done loading
+                          }
+                      })
+                      fileAudio.audio.addEventListener('error', function(e) {
+                          restart(function(MACHINE) {
+                              plt.baselib.exceptions.raiseFailure(
+                                  MACHINE,
+                                  plt.baselib.format.format(
+                                      "unable to load ~a: ~a",
+                                      [path,
+                                      e.message]));
+                              });
+                      });
+                  });
+            }
+        });
+
 
 
 EXPORTS['overlay'] = 
@@ -1398,4 +1448,4 @@ EXPORTS['name->color'] =
             var name = checkSymbolOrString(MACHINE, 'name->color', 0);
             var result = colorDb.get('' + name) || false;
             return result;
-        });
+        });

whalesong/image/private/kernel.js

@@ -456,7 +456,7 @@ FileImage.prototype.equals = function(other, aUnionFind) {
 };
 
 //////////////////////////////////////////////////////////////////////
-// FileVideoe: String Node -> Video
+// FileVideo: String Node -> Video
 var FileVideo = function(src, rawVideo) {
     BaseImage.call(this);
     var self = this;
@@ -507,52 +507,39 @@ FileVideo.prototype.render = function(ctx, x, y) {
     ctx.drawImage(this.video, x, y);
 };
 FileVideo.prototype.equals = function(other, aUnionFind) {
-  return (other instanceof FileVideo) && (this.src === other.src);
+    return (other instanceof FileVideo) && (this.src === other.src);
 };
 
 //////////////////////////////////////////////////////////////////////
-// FileAudio: String Node -> Video
+// FileAudio: String Boolean Audio -> FileAudio
 var FileAudio = function(src, loop, rawAudio) {
-  this.src = src;
+    this.src = src;
-  var that = this;
+    if(rawAudio) {
-  if (rawAudio && (rawAudio.readyState===4)) {
+        this.audio = rawAudio;
-    that.audio                  = rawAudio;
+    }
-    that.audio.autoplay         = true;
+    else {
-    that.audio.autobuffer       = true;
+        this.audio = document.createElement('audio');
-    that.audio.currentTime      = 0;
+    }
-    that.audio.loop             = loop;
+    this.audio.src = src;
-    that.audio.play();
+    this.audio.autoplay         = false;
-  } else {
+    this.audio.autobuffer       = true;
-    // fixme: we may want to do something blocking here for
+    // the below caused firefox to hang (but things work ok without it)
-    // onload, since we don't know at this time what the file size
+    // this.audio.currentTime      = 0;
-    // should be, nor will drawImage do the right thing until the
+    this.audio.loop             = loop;
-    // file is loaded.
+    return true;
-    that.audio = document.createElement('audio');
-    that.audio.src = src;
-    that.audio.addEventListener('canplay', function() {
-                                that.audio.autoplay     = true;
-                                that.audio.autobuffer   = true;
-                                that.audio.currentTime  = 0;
-                                that.audio.loop         = loop;
-                                that.audio.play();
-                                });
-  }
-  return true;
 };
-var audioCache = {};
+
 FileAudio.makeInstance = function(path, loop, rawAudio) {
-  /*        if (! (path in audioCache)) {
+    return new FileAudio(path, loop, rawAudio);
-   audioCache[path] = new FileAudio(path, loop, rawAudio, afterInit);
-   return audioCache[path];
-   } else {
-   audioCache[path].audio.play();
-   afterInit(audioCache[path]);
-   return audioCache[path];
-   }
-   */
-  return new FileAudio(path, loop, rawAudio);
 };
 
+FileAudio.prototype.play = function () {
+    this.audio.play();
+    return true;
+}
+
+
+
 //////////////////////////////////////////////////////////////////////
 // ImageDataImage: imageData -> image
 // Given an array of pixel data, create an image
@@ -1384,8 +1371,8 @@ var makeFileImage = function(path, rawImage) {
 var makeFileVideo = function(path, rawVideo) {
     return FileVideo.makeInstance(path, rawVideo);
 };
-var makeFileAudio = function(path, rawAudio){
+var makeFileAudio = function(path, loop, rawAudio){
-  return FileAudio.makeInstance(path, rawAudio)
+  return FileAudio.makeInstance(path, loop, rawAudio)
 };
 
 
@@ -1409,7 +1396,7 @@ var isFlipImage	= function(x) { return x instanceof FlipImage; };
 var isTextImage	= function(x) { return x instanceof TextImage; };
 var isFileImage	= function(x) { return x instanceof FileImage; };
 var isFileVideo	= function(x) { return x instanceof FileVideo; };
-
+var isFileAudio = function(x) { return x instanceof FileAudio; };
 
 
 
@@ -1503,7 +1490,7 @@ EXPORTS.isFlipImage = isFlipImage;
 EXPORTS.isTextImage = isTextImage;
 EXPORTS.isFileImage = isFileImage;
 EXPORTS.isFileVideo = isFileVideo;
-//EXPORTS.isFileAudio = isFileAudio;
+EXPORTS.isFileAudio = isFileAudio;
 
 
 EXPORTS.makeColor = makeColor;