Ah-ha. After testing id3 tags with 3 different mp3 files that all

should have had id3 tags, and trying everything, I tried one
more mp3 file: bingo.
Added MediaPlayer.getMetadata() and a METADATA event.
Contents of metadata vary by media being played and are open for
interpretation by users of the player.
FLV video returns their metadata object.
MP3 audio returns id3 tags.


git-svn-id: svn+ssh://src.earth.threerings.net/nenya/trunk@708 ed5b42cb-e716-0410-a449-f6a68f950b19

src/as/com/threerings/flash/media/FlvVideoPlayer.as

@@ -151,9 +151,16 @@ public class FlvVideoPlayer extends EventDispatcher
         }
     }
 
+    // from VideoPlayer
+    public function getMetadata () :Object
+    {
+        return _metadata;
+    }
+
     // from VideoPlayer
     public function unload () :void
     {
+        _metadata = null;
         _sizeChecker.reset();
         _positionChecker.reset();
         _vid.attachNetStream(null);
@@ -266,17 +273,15 @@ public class FlvVideoPlayer extends EventDispatcher
      */
     protected function handleMetaData (info :Object) :void
     {
-//        log.debug("Got video metadata:");
-//        for (var n :String in info) {
-//            log.debug("    " + n + ": " + info[n]);
-//        }
-
         if ("duration" in info) {
             _duration = Number(info.duration);
             if (!isNaN(_duration)) {
                 dispatchEvent(new ValueEvent(MediaPlayerCodes.DURATION, _duration));
             }
         }
+
+        _metadata = info;
+        dispatchEvent(new ValueEvent(MediaPlayerCodes.METADATA, info));
     }
 
     protected function updateState (newState :int) :void
@@ -316,6 +321,8 @@ public class FlvVideoPlayer extends EventDispatcher
 
     protected var _volume :Number = 1;
 
+    protected var _metadata :Object;
+
     /** Checks the video every 100ms to see if the dimensions are now know.
      * Yes, this is how to do it. We could trigger on ENTER_FRAME, but then
      * we may not know the dimensions unless we're added on the display list,

src/as/com/threerings/flash/media/MediaPlayer.as

@@ -32,6 +32,14 @@ import com.threerings.util.ValueEvent;
  */
 [Event(name="position", type="com.threerings.util.ValueEvent")]
 
+/**
+ * Dispatched when media metadata is available, if ever.
+ * <b>value</b>: metadata object. @see getMetadata() for more info.
+ *
+ * @eventType com.threerings.flash.media.MediaPlayerCodes.METADATA
+ */
+[Event(name="metadata", type="com.threerings.util.ValueEvent")]
+
 /**
  * Dispatched when there's a problem.
  * <b>value</b>: a String error code/message.
@@ -85,6 +93,12 @@ public interface MediaPlayer extends IEventDispatcher
      */
     function getVolume () :Number;
 
+    /**
+     * Get metadata, or null if none or not yet available. The exact format of this data
+     * object varies by MediaPlayer implementation, see the documentation for each.
+     */
+    function getMetadata () :Object;
+
     /**
      * Unload the media.
      */

src/as/com/threerings/flash/media/MediaPlayerCodes.as

@@ -11,6 +11,8 @@ public class MediaPlayerCodes
 
     public static const POSITION :String = "position";
 
+    public static const METADATA :String = "metadata";
+
     /** Only applicable for VideoPlayer. */
     public static const SIZE :String = "size";
 

src/as/com/threerings/flash/media/Mp3AudioPlayer.as

@@ -117,6 +117,12 @@ public class Mp3AudioPlayer extends EventDispatcher
         return _volume;
     }
 
+    // from AudioPlayer
+    public function getMetadata () :Object
+    {
+        return (_sound == null) ? null : _sound.id3;
+    }
+
     // from AudioPlayer
     public function unload () :void
     {
@@ -132,6 +138,9 @@ public class Mp3AudioPlayer extends EventDispatcher
         _lastPosition = NaN;
     }
 
+    /**
+     * Play without updating the current state.
+     */
     protected function play0 () :void
     {
         pause0();
@@ -143,6 +152,9 @@ public class Mp3AudioPlayer extends EventDispatcher
         }
     }
 
+    /**
+     * Pause without saving the position or updating the state.
+     */
     protected function pause0 () :void
     {
         if (_chan != null) {
@@ -177,7 +189,7 @@ public class Mp3AudioPlayer extends EventDispatcher
 
     protected function handleId3 (event :Event) :void
     {
-        trace("Got ID3");
+        dispatchEvent(new ValueEvent(MediaPlayerCodes.METADATA, _sound.id3));
     }
 
     /**