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

Side by Side Diff: media/ffmpeg/ffmpeg_unittest.cc

Issue 3043049: Adding two test cases to ffmpeg_unittest from user bug reports. (Closed)
Patch Set: Adding comments Created 10 years, 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // ffmpeg_unittests verify that the parts of the FFmpeg API that Chromium uses 5 // ffmpeg_unittests verify that the parts of the FFmpeg API that Chromium uses
6 // function as advertised for each media format that Chromium supports. This 6 // function as advertised for each media format that Chromium supports. This
7 // mostly includes stuff like reporting proper timestamps, seeking to 7 // mostly includes stuff like reporting proper timestamps, seeking to
8 // keyframes, and supporting certain features like reordered_opaque. 8 // keyframes, and supporting certain features like reordered_opaque.
9 // 9 //
10 // Known failures as of r53892: 10 // Known failures as of r54591:
11 // http://crbug.com/47761
12 // crbug47761_ogg/FFmpegTest.Loop_Audio/0
13 // crbug47761_ogg/FFmpegTest.Seek_Audio/0
14 // crbug47761_ogg/FFmpegTest.Decode_Audio/0
15 //
11 // http://crbug.com/49709 16 // http://crbug.com/49709
12 // sync1_ogg/FFmpegTest.Seek_Audio/0 17 // sync1_ogg/FFmpegTest.Seek_Audio/0
13 // sync1_ogv/FFmpegTest.Seek_Audio/0 18 // sync1_ogv/FFmpegTest.Seek_Audio/0
14 // sync2_ogg/FFmpegTest.Seek_Audio/0 19 // sync2_ogg/FFmpegTest.Seek_Audio/0
15 // sync2_ogv/FFmpegTest.Seek_Audio/0 20 // sync2_ogv/FFmpegTest.Seek_Audio/0
16 // 21 //
17 // http://crbug.com/50457 22 // http://crbug.com/50457
18 // sync0_webm/FFmpegTest.Decode_Video/0 23 // sync0_webm/FFmpegTest.Decode_Video/0
19 // sync0_webm/FFmpegTest.Duration/0 24 // sync0_webm/FFmpegTest.Duration/0
20 // sync1_webm/FFmpegTest.Decode_Video/0 25 // sync1_webm/FFmpegTest.Decode_Video/0
(...skipping 419 matching lines...) Expand 10 before | Expand all | Expand 10 after
440 FFMPEG_TEST_CASE(sync2, m4a); 445 FFMPEG_TEST_CASE(sync2, m4a);
441 FFMPEG_TEST_CASE(sync2, mp3); 446 FFMPEG_TEST_CASE(sync2, mp3);
442 FFMPEG_TEST_CASE(sync2, mp4); 447 FFMPEG_TEST_CASE(sync2, mp4);
443 FFMPEG_TEST_CASE(sync2, ogg); 448 FFMPEG_TEST_CASE(sync2, ogg);
444 FFMPEG_TEST_CASE(sync2, ogv); 449 FFMPEG_TEST_CASE(sync2, ogv);
445 FFMPEG_TEST_CASE(sync2, webm); 450 FFMPEG_TEST_CASE(sync2, webm);
446 451
447 // Covers our LayoutTest file. 452 // Covers our LayoutTest file.
448 FFMPEG_TEST_CASE(counting, ogv); 453 FFMPEG_TEST_CASE(counting, ogv);
449 454
455 // The following are bugs reported by users.
456 FFMPEG_TEST_CASE(crbug47761, ogg);
457 FFMPEG_TEST_CASE(crbug50045, mp4);
458
450 TEST_P(FFmpegTest, Perf) { 459 TEST_P(FFmpegTest, Perf) {
451 { 460 {
452 PerfTimeLogger timer("Opening file"); 461 PerfTimeLogger timer("Opening file");
453 OpenFile(GetParam()); 462 OpenFile(GetParam());
454 } 463 }
455 { 464 {
456 PerfTimeLogger timer("Opening codecs"); 465 PerfTimeLogger timer("Opening codecs");
457 OpenCodecs(); 466 OpenCodecs();
458 } 467 }
459 { 468 {
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
637 SeekTo(0.4); 646 SeekTo(0.4);
638 ReadRemainingFile(); 647 ReadRemainingFile();
639 EXPECT_TRUE(StepDecodeVideo()); 648 EXPECT_TRUE(StepDecodeVideo());
640 LOG(INFO) << decoded_video_time(); 649 LOG(INFO) << decoded_video_time();
641 650
642 CloseCodecs(); 651 CloseCodecs();
643 CloseFile(); 652 CloseFile();
644 } 653 }
645 654
646 } // namespace media 655 } // namespace media
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698