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

Unified Diff: media/filters/source_buffer_stream_unittest.cc

Issue 213253006: MSE: Populate WebM missing duration with DefaultDuration, derived, or default (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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/source_buffer_stream_unittest.cc
diff --git a/media/filters/source_buffer_stream_unittest.cc b/media/filters/source_buffer_stream_unittest.cc
index a41fb9c6f726497a9a479c9fa6a2b3fa54f4f8bb..a35302c7e9f9e49b3e3c623a53a0439b91773d3e 100644
--- a/media/filters/source_buffer_stream_unittest.cc
+++ b/media/filters/source_buffer_stream_unittest.cc
@@ -3621,7 +3621,9 @@ TEST_F(SourceBufferStreamTest,
CheckNoNextBuffer();
}
-TEST_F(SourceBufferStreamTest, Audio_SpliceFrame_Basic) {
+// TODO(dalecurtis): Re-enable once GenerateSpliceFrame() is re-enabled. See
+// http://crbug.com/356805.
wolenetz 2014/03/27 18:01:48 Likewise for these tests, I'm undoing my disabling
+TEST_F(SourceBufferStreamTest, DISABLED_Audio_SpliceFrame_Basic) {
SetAudioStream();
Seek(0);
NewSegmentAppend("0K 2K 4K 6K 8K 10K 12K");
@@ -3630,7 +3632,10 @@ TEST_F(SourceBufferStreamTest, Audio_SpliceFrame_Basic) {
CheckNoNextBuffer();
}
-TEST_F(SourceBufferStreamTest, Audio_SpliceFrame_DoubleSpliceExactOverlap) {
+// TODO(dalecurtis): Re-enable once GenerateSpliceFrame() is re-enabled. See
+// http://crbug.com/356805.
+TEST_F(SourceBufferStreamTest,
+ DISABLED_Audio_SpliceFrame_DoubleSpliceExactOverlap) {
SetAudioStream();
Seek(0);
NewSegmentAppend("0K 2K 4K 6K 8K 10K 12K");
@@ -3643,7 +3648,10 @@ TEST_F(SourceBufferStreamTest, Audio_SpliceFrame_DoubleSpliceExactOverlap) {
CheckNoNextBuffer();
}
-TEST_F(SourceBufferStreamTest, Audio_SpliceFrame_DoubleSpliceEarlierOverlap) {
+// TODO(dalecurtis): Re-enable once GenerateSpliceFrame() is re-enabled. See
+// http://crbug.com/356805.
+TEST_F(SourceBufferStreamTest,
+ DISABLED_Audio_SpliceFrame_DoubleSpliceEarlierOverlap) {
SetAudioStream();
Seek(0);
NewSegmentAppend("0K 2K 4K 6K 8K 10K 12K");
@@ -3659,7 +3667,10 @@ TEST_F(SourceBufferStreamTest, Audio_SpliceFrame_DoubleSpliceEarlierOverlap) {
CheckNoNextBuffer();
}
-TEST_F(SourceBufferStreamTest, Audio_SpliceFrame_DoubleSpliceLaterOverlap) {
+// TODO(dalecurtis): Re-enable once GenerateSpliceFrame() is re-enabled. See
+// http://crbug.com/356805.
+TEST_F(SourceBufferStreamTest,
+ DISABLED_Audio_SpliceFrame_DoubleSpliceLaterOverlap) {
SetAudioStream();
Seek(0);
NewSegmentAppend("0K 2K 4K 6K 8K 10K 12K");

Powered by Google App Engine
This is Rietveld 408576698