Add "type" in GenerateKeyRequest() and OnNeedKey().

media/webm/webm_stream_parser.cc

Index: media/webm/webm_stream_parser.cc
diff --git a/media/webm/webm_stream_parser.cc b/media/webm/webm_stream_parser.cc
index 2e34ed96c80885831af4c1a9d133da694d2c9ca9..8cf09999f02b52ab6987720721f00fcf7576c610 100644
--- a/media/webm/webm_stream_parser.cc
+++ b/media/webm/webm_stream_parser.cc
@@ -19,6 +19,9 @@
 
 namespace media {
 
+// TODO(xhwang): Figure out the init data type appropriately once it's spec'ed.
+static const char kWebMInitDataType[] = "video/webm";
+
 // Helper class that uses FFmpeg to create AudioDecoderConfig &
 // VideoDecoderConfig objects.
 //
@@ -474,7 +477,7 @@ void WebMStreamParser::FireNeedKey(const std::string& key_id) {
   DCHECK_GT(key_id_size, 0);
   scoped_array<uint8> key_id_array(new uint8[key_id_size]);
   memcpy(key_id_array.get(), key_id.data(), key_id_size);
-  need_key_cb_.Run(key_id_array.Pass(), key_id_size);
+  need_key_cb_.Run(kWebMInitDataType, key_id_array.Pass(), key_id_size);

[ddorwin, 2012/10/27 01:47:17]

Is there no test for the WebM parser like there is for MP4? Or do we just have
the not empty check in chunk_demuxer_unittest.cc? Is webm_parser_unittest.cc –
the equivalent of the mp4 test?

[xhwang, 2012/10/27 03:22:59]

It's pretty much tested in chunk_demuxer_unittests where we actually use webm
for testing (for historical reasons). Added check for "video/webm" there.

 }
 
 }  // namespace media