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

Unified Diff: media/filters/pipeline_integration_test.cc

Issue 10829470: Support for parsing encrypted WebM streams by src. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Refactored decryption code. Add 2 unittests. Created 7 years, 9 months 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/filters/pipeline_integration_test.cc
diff --git a/media/filters/pipeline_integration_test.cc b/media/filters/pipeline_integration_test.cc
index da8b26f5c2af0f9b4fdbb2f81f81d76848203799..ac09501e4bbd99d7231473fcb1b82a98bcefb405 100644
--- a/media/filters/pipeline_integration_test.cc
+++ b/media/filters/pipeline_integration_test.cc
@@ -313,6 +313,7 @@ class MockMediaSource {
scoped_array<uint8> init_data, int init_data_size) {
DCHECK(init_data.get());
DCHECK_GT(init_data_size, 0);
+ ASSERT_FALSE(need_key_cb_.is_null());
need_key_cb_.Run("", "", type, init_data.Pass(), init_data_size);
}
@@ -424,6 +425,22 @@ TEST_F(PipelineIntegrationTest, BasicPlaybackHashed) {
// EXPECT_EQ(GetAudioHash(), "");
}
+// TODO(fgalligan): Enable test when code to parse encrypted WebM files lands
+// in Chromium's FFmpeg.
+TEST_F(PipelineIntegrationTest, DISABLED_BasicPlaybackEncrypted) {
+ FakeEncryptedMedia encrypted_media(new KeyProvidingApp());
+ set_need_key_cb(base::Bind(&FakeEncryptedMedia::NeedKey,
+ base::Unretained(&encrypted_media)));
+
+ ASSERT_TRUE(Start(GetTestDataFilePath("bear-640x360-av_enc-av.webm"),
+ encrypted_media.decryptor()));
+
+ Play();
+
+ ASSERT_TRUE(WaitUntilOnEnded());
+ Stop();
+}
+
TEST_F(PipelineIntegrationTest, BasicPlayback_MediaSource) {
MockMediaSource source("bear-320x240.webm", kWebM, 219229);
StartPipelineWithMediaSource(&source);

Powered by Google App Engine
This is Rietveld 408576698