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

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

Issue 2563183002: Fix up missing timestamps in FFmpegDemuxer. (Closed)
Patch Set: added demuxer status to media log Created 4 years 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_demuxer.h » ('j') | media/filters/ffmpeg_demuxer.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // Regression tests for FFmpeg. Test files can be found in the internal media 5 // Regression tests for FFmpeg. Test files can be found in the internal media
6 // test data directory: 6 // test data directory:
7 // 7 //
8 // https://chrome-internal.googlesource.com/chrome/data/media 8 // https://chrome-internal.googlesource.com/chrome/data/media
9 // 9 //
10 // Simply add the custom_dep below to your gclient and sync: 10 // Simply add the custom_dep below to your gclient and sync:
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 PIPELINE_ERROR_INITIALIZATION_FAILED); 187 PIPELINE_ERROR_INITIALIZATION_FAILED);
188 FFMPEG_TEST_CASE(Cr640889, "security/640889.flac", PIPELINE_OK, PIPELINE_OK); 188 FFMPEG_TEST_CASE(Cr640889, "security/640889.flac", PIPELINE_OK, PIPELINE_OK);
189 FFMPEG_TEST_CASE(Cr640912, "security/640912.flac", PIPELINE_OK, PIPELINE_OK); 189 FFMPEG_TEST_CASE(Cr640912, "security/640912.flac", PIPELINE_OK, PIPELINE_OK);
190 // TODO(liberato): before crbug.com/658440 was fixed, this would fail if run 190 // TODO(liberato): before crbug.com/658440 was fixed, this would fail if run
191 // twice under ASAN. If run once, then it doesn't. However, it still catches 191 // twice under ASAN. If run once, then it doesn't. However, it still catches
192 // issues in crbug.com/662118, so it's included anyway. 192 // issues in crbug.com/662118, so it's included anyway.
193 FFMPEG_TEST_CASE(Cr658440, "security/658440.flac", PIPELINE_OK, PIPELINE_OK); 193 FFMPEG_TEST_CASE(Cr658440, "security/658440.flac", PIPELINE_OK, PIPELINE_OK);
194 FFMPEG_TEST_CASE(Cr665305, 194 FFMPEG_TEST_CASE(Cr665305,
195 "crbug665305.flac", 195 "crbug665305.flac",
196 PIPELINE_OK, 196 PIPELINE_OK,
197 PIPELINE_ERROR_DECODE); 197 DEMUXER_ERROR_COULD_NOT_PARSE);
198 FFMPEG_TEST_CASE_SEEKING(Cr666770, 198 FFMPEG_TEST_CASE_SEEKING(Cr666770,
199 "security/666770.mp4", 199 "security/666770.mp4",
200 PIPELINE_OK, 200 PIPELINE_OK,
201 PIPELINE_OK, 201 PIPELINE_OK,
202 base::TimeDelta::FromSecondsD(0.0843)); 202 base::TimeDelta::FromSecondsD(0.0843));
203 FFMPEG_TEST_CASE(Cr666874, 203 FFMPEG_TEST_CASE(Cr666874,
204 "security/666874.mp3", 204 "security/666874.mp3",
205 DEMUXER_ERROR_COULD_NOT_OPEN, 205 DEMUXER_ERROR_COULD_NOT_OPEN,
206 DEMUXER_ERROR_COULD_NOT_OPEN); 206 DEMUXER_ERROR_COULD_NOT_OPEN);
207 FFMPEG_TEST_CASE(Cr667063, "security/667063.mp4", PIPELINE_OK, PIPELINE_OK); 207 FFMPEG_TEST_CASE(Cr667063, "security/667063.mp4", PIPELINE_OK, PIPELINE_OK);
(...skipping 207 matching lines...) Expand 10 before | Expand all | Expand 10 after
415 } 415 }
416 416
417 TEST_P(FlakyFFmpegRegressionTest, BasicPlayback) { 417 TEST_P(FlakyFFmpegRegressionTest, BasicPlayback) {
418 if (Start(GetParam().filename, kClockless) == PIPELINE_OK) { 418 if (Start(GetParam().filename, kClockless) == PIPELINE_OK) {
419 Play(); 419 Play();
420 WaitUntilEndedOrError(); 420 WaitUntilEndedOrError();
421 } 421 }
422 } 422 }
423 423
424 } // namespace media 424 } // namespace media
OLDNEW
« no previous file with comments | « no previous file | media/filters/ffmpeg_demuxer.h » ('j') | media/filters/ffmpeg_demuxer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698