Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(672)

Unified Diff: media/mojo/common/mojo_decoder_buffer_converter_unittest.cc

Issue 2484763005: media: Support transporting 0-byte DecoderBuffer over mojo (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/mojo/common/mojo_decoder_buffer_converter_unittest.cc
diff --git a/media/mojo/common/mojo_decoder_buffer_converter_unittest.cc b/media/mojo/common/mojo_decoder_buffer_converter_unittest.cc
index 6d998e4f69d8ab3e500c9d23624bff5d868a1724..11b6c50b4fcfb8e7b16ec9fa0795c2a4efd77401 100644
--- a/media/mojo/common/mojo_decoder_buffer_converter_unittest.cc
+++ b/media/mojo/common/mojo_decoder_buffer_converter_unittest.cc
@@ -96,6 +96,16 @@ TEST(MojoDecoderBufferConverterTest, ConvertDecoderBuffer_EOS) {
converter.ConvertAndVerify(buffer);
}
+// TODO(xhwang): Investigate whether we can get rid of zero-byte-buffer.
+// See http://crbug.com/663438
+TEST(MojoDecoderBufferConverterTest, ConvertDecoderBuffer_ZeroByteBuffer) {
+ base::MessageLoop message_loop;
+ scoped_refptr<DecoderBuffer> buffer(new DecoderBuffer(0));
+
+ MojoDecoderBufferConverter converter;
+ converter.ConvertAndVerify(buffer);
+}
+
TEST(MojoDecoderBufferConverterTest, ConvertDecoderBuffer_KeyFrame) {
base::MessageLoop message_loop;
const uint8_t kData[] = "hello, world";

Powered by Google App Engine
This is Rietveld 408576698