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

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

Issue 3077004: Updated some comments to point at bugs in ffmpeg_unittest.cc and ffmpeg_audio_decoder.cc. (Closed)
Patch Set: Created 10 years, 5 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 | media/filters/ffmpeg_audio_decoder.cc » ('j') | 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
6 // function as advertised for each media format that Chromium supports. This
7 // mostly includes stuff like reporting proper timestamps, seeking to
8 // keyframes, and supporting certain features like reordered_opaque.
9 //
10 // Known failures as of r53892:
11 // http://crbug.com/49709
12 // sync1_ogg/FFmpegTest.Seek_Audio/0
13 // sync1_ogv/FFmpegTest.Seek_Audio/0
14 // sync2_ogg/FFmpegTest.Seek_Audio/0
15 // sync2_ogv/FFmpegTest.Seek_Audio/0
16 //
17 // http://crbug.com/50457
18 // sync0_webm/FFmpegTest.Decode_Video/0
19 // sync0_webm/FFmpegTest.Duration/0
20 // sync1_webm/FFmpegTest.Decode_Video/0
21 // sync2_webm/FFmpegTest.Decode_Video/0
22
5 #include <limits> 23 #include <limits>
6 #include <queue> 24 #include <queue>
7 25
8 #include "base/base_paths.h" 26 #include "base/base_paths.h"
9 #include "base/file_path.h" 27 #include "base/file_path.h"
10 #include "base/file_util.h" 28 #include "base/file_util.h"
11 #include "base/path_service.h" 29 #include "base/path_service.h"
12 #include "base/perftimer.h" 30 #include "base/perftimer.h"
13 #include "base/string_util.h" 31 #include "base/string_util.h"
14 #include "base/test/perf_test_suite.h" 32 #include "base/test/perf_test_suite.h"
(...skipping 604 matching lines...) Expand 10 before | Expand all | Expand 10 after
619 SeekTo(0.4); 637 SeekTo(0.4);
620 ReadRemainingFile(); 638 ReadRemainingFile();
621 EXPECT_TRUE(StepDecodeVideo()); 639 EXPECT_TRUE(StepDecodeVideo());
622 LOG(INFO) << decoded_video_time(); 640 LOG(INFO) << decoded_video_time();
623 641
624 CloseCodecs(); 642 CloseCodecs();
625 CloseFile(); 643 CloseFile();
626 } 644 }
627 645
628 } // namespace media 646 } // namespace media
OLDNEW
« no previous file with comments | « no previous file | media/filters/ffmpeg_audio_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698