Media Metadata API: Extract raw tags for audio/video.

chrome/common/extensions/api/media_galleries.idl

Index: chrome/common/extensions/api/media_galleries.idl
diff --git a/chrome/common/extensions/api/media_galleries.idl b/chrome/common/extensions/api/media_galleries.idl
index ce645e82a765577290a9b79b545c2828a4545d87..941c7b786e368bb3668f5ca205cbb11c9e80fdc6 100644
--- a/chrome/common/extensions/api/media_galleries.idl
+++ b/chrome/common/extensions/api/media_galleries.idl
@@ -120,6 +120,10 @@ namespace mediaGalleries {
     DOMString? language;
     DOMString? title;
     long? track;
+
+    // A dictionary of all the metadata in the media file. This metadata is in
+    // string form, and neither the key nor value is normalized.
+    object rawTags;

[Lei Zhang, 2014/04/01 19:28:10]

Maybe there is a better type for this? The extensions folks may know.

[tommycli, 2014/04/01 19:29:01]

yoz: Is 'object' the right data type to use? I essentially a map<string, string>
with no pre-defined keys.

[Yoyo Zhou, 2014/04/01 21:39:40]

I'm not that familiar with IDL. Adding asargent.

[asargent_no_longer_on_chrome, 2014/04/02 20:03:54]

Yeah, if you don't know the set of possible keys up front, it is appropriate to
use 'object'.

If there are some keys that you do know in advance and want to explicitly call
those out in documentation, but leave room for future expansion, one thing you
might do is:

dictionary Tags {
  long? foo;
  DOMSTRING? bar;
  object? additionalTags;
};


A better way to leave room for future expansion in WebIDL, that I think our
parser unfortunately does not currently support, would be:

interface Tags {
  attribute? long foo;
  attribute? DOMString bar;
}


This specifies the foo and bar attributes as things that are known in the
interface, but does not preclude additional named properties by objects
implementing the Tags interface. If adding this support sounds like fun, I'd be
happy to review CL's! But if you want to avoid the yak shave, going with just
object or the dictionary-with-embedded-object approaches are fine. 

   };
 
   callback MediaMetadataCallback = void (MediaMetadata metadata);